-
Notifications
You must be signed in to change notification settings - Fork 15
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
Update readme #85
Update readme #85
Conversation
mohamedsalem401
commented
Dec 5, 2023
- Add discord link
- Explain how to add Custom fields
- Explain how to add Configuration
|
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.
Really like reviewing by README PR 👏👏
Whilst i think there may be some changes we could make this is good to approve and we discuss a bit more.
key3: 123, | ||
key4: ["a", "b", "c"] // this will match exactly ["a", "b", "c"] | ||
} | ||
frontmatter: { |
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.
Can we do style guide stuff separately if we can - makes it easier to understand the PR.
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.
OK
README.md
Outdated
@@ -113,6 +115,70 @@ const blogs = await mddb.getFiles({ | |||
}); | |||
``` | |||
|
|||
# Configuring `mddb.config.js` |
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.
What happened to our simple code config in the first version. I'd do that first ...
Overall this approach seems pretty sound and we would need it at some point. I'd just like to get the details right with some code before having to do config stuff.