-
Notifications
You must be signed in to change notification settings - Fork 12
fix: update rdme now that this is on superhub #1061
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: next
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,14 +1,17 @@ | ||
--- | ||
title: 'Callouts' | ||
category: 5fdf7610134322007389a6ed | ||
hidden: false | ||
title: Callouts | ||
category: | ||
uri: syntax | ||
metadata: | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. have you tried syncing locally? i thiiiink this There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Oof yeah I tried this and it broke the markdown repo really bad (with no warning). I had to setup bidi and do a git revert from there to fix it. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 😬 yikes okay let me know if you need a hand! and if you have more info on what the breakage was, i'm happy to look into it! our API is rather strict about request payloads now so i would have expected it to throw rather than accept this and break stuff 😮💨 for add'l background, with the end of january sprint on this There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is the revert commit! https://github.com/readmeio/rdmd-docs/commit/8a1c29f0075e3a6eb3aa7b539fab2dec1a3cef0d The big thing I think was the Syntax and syntax folders that got created for some reason. I also think some of the frontmatter that got created might have been invalid? Not sure There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. ah okay — none of the diff looks frontmatter-related so not sure. i suppose this repo is pretty much the worst case scenario when it comes to MDX issues since the whole point of these docs is to demonstrate and test the limits of our markdown support 🫠 we deemed MDX issues out-of-scope for this |
||
image: | ||
0: 'https://files.readme.io/9134da7-rdmd.readme-stage-pr-2438.readme.ninja_docs_callouts.png' | ||
1: 'rdmd.readme-stage-pr-2438.readme.ninja_docs_callouts.png' | ||
2: 530 | ||
3: 173 | ||
4: '#edf3ec' | ||
'0': >- | ||
https://files.readme.io/9134da7-rdmd.readme-stage-pr-2438.readme.ninja_docs_callouts.png | ||
'1': rdmd.readme-stage-pr-2438.readme.ninja_docs_callouts.png | ||
'2': 530 | ||
'3': 173 | ||
'4': '#edf3ec' | ||
privacy: | ||
view: public | ||
--- | ||
|
||
## Syntax | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i heard rumblings that we're consolidating these docs into the main project but i believe this API key points to the markdown project!