-
Notifications
You must be signed in to change notification settings - Fork 13
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
docs(docs): 📝 update about information #637
Conversation
This branch is running in CodeSandbox. Use the links below to review this PR faster. |
✅ Deploy Preview for ripple-docs ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
layout: page | ||
--- | ||
|
||
This is how you do that |
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.
@waitingallday can I get you to please write this guide? Branch from this branch and write up how to create a new content type in Ripple 2 - leave out the layers stuff, that is covered in #662
|
||
Please review the following concepts before proceeding: | ||
|
||
- [Content types](../2.key-concepts/4.content-types.md) |
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.
@dylankelly just a heads up you don't need to add the numbers for URLs. It's probably better to them off too so that changing the order doesn't break the link
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.
@jeffdowdle its a link to the file not the URL - I have a VS extension to add them in easily, means you don't have to think about the URL and renaming the file can be dealt with via find / replace. But yeah, should probably be consistent lets discuss the best option.
tweak imports, make error imports consistant
Issue:
What I did
How to test
Checklist
For all PR's
For new components only