-
Notifications
You must be signed in to change notification settings - Fork 85
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
WIP - Blog extensions: Reviews and Ratings #861
base: main
Are you sure you want to change the base?
Conversation
Tagging OptionsShould a new tag be published when this PR is merged?
|
Feat blog comment
Feat blog comment
Feat blog extesions
@aka-sacci-ccr this seems as a big PR / contribution. Can you describe your intent and the work that is still pending on an issue? |
@matheusgr Basically, it's just two extensions in the blog loaders, using deco records to save these interactions. |
blog/utils/records.ts
Outdated
): Promise<Review[]> { | ||
const records = await ctx.invoke.records.loaders.drizzle(); | ||
|
||
const whereClause = ignoreReviews?.active && ignoreReviews?.markedAs && |
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.
Use the name utils
for things like: checkIfDateIsBefore
, encodeToken
.. and not your core business logic.
We don't have something well setup for that, but you'll come up with a good name
What is this Contribution About?
With this contribution, it is possible to add extensions to blog loaders.
The two extensions created enable user interactions with posts - Ratings and Reviews
Loom Video
Demonstration Link