-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add Entry Tags #202
base: v3
Are you sure you want to change the base?
Add Entry Tags #202
Conversation
I'm going to work on this tommorow with @JetStream0 as i did the most that i can do myself with the database. On Sunday, i will work on the Edit Entry design, so people can add Tags into their Entry |
We will need to add a search bar with tag search support, and a way for admins to add tags. We might want to consider adding tag search to the discord bot. |
This is still a work in progress, but is ready for some feedback. Please review our code and test it so we can fix any issues that you may detect! |
shit, completely forgot about needing admins to add tags. imma need your help, like a lot since idk shit about mongoose on typescript |
Will add tag search into discord bot sometime, maybe we could add it as a separate field but make it optional |
I believe we are ready for review. Everything should be done and probably working. @mat-1 |
Co-authored-by: mat <27899617+mat-1@users.noreply.github.com>
Co-authored-by: mat <27899617+mat-1@users.noreply.github.com>
Co-authored-by: mat <27899617+mat-1@users.noreply.github.com>
Co-authored-by: mat <27899617+mat-1@users.noreply.github.com>
Removed "Ready for Review" label as this cannot be reviewed due to it failing the checks. Please resolve all the conversations as well as the conflicts. |
Fixed some stuff, need better review now
This pr is now ready for review, and the "Ready for Review" label has been added. I have also removed @mat-1's review as well as conversations/review comments. Please make sure this pr is reviewed closely and ensure that it won't make any breaking changes toward production. |
This is still awaiting review from @mat-1 |
Co-authored-by: mat <27899617+mat-1@users.noreply.github.com>
Co-authored-by: mat <27899617+mat-1@users.noreply.github.com>
Co-authored-by: mat <27899617+mat-1@users.noreply.github.com>
3547c0c
to
271549f
Compare
@Codemonkey51 Please review! Tested and working. |
This will add Entry Tags into Repldex v3!