search

repldex / Repldex

https://repldex.com
MIT License
21 stars 10 forks source link

Add Entry Tags #202

Open selectdev opened 2 years ago

selectdev commented 2 years ago

This will add Entry Tags into Repldex v3!

selectdev commented 2 years ago

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

stjet commented 2 years ago

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.

selectdev commented 2 years ago

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!

selectdev commented 2 years ago

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.

shit, completely forgot about needing admins to add tags. imma need your help, like a lot since idk shit about mongoose on typescript

selectdev commented 2 years ago

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.

Will add tag search into discord bot sometime, maybe we could add it as a separate field but make it optional

stjet commented 2 years ago

I believe we are ready for review. Everything should be done and probably working. @mat-1

selectdev commented 2 years ago

I guess, this can be merged. Nobody said anything about adding entry tags into history, so should be good

selectdev commented 2 years ago

idk why this still exists, but i guess imma start working on it again since why not. for right now, i'm going to set this pr to draft

stjet commented 2 years ago

remember me forgetting to do the window.location thing with the <div>

Yeah lol

selectdev commented 2 years ago

This pr is ready for review!

selectdev commented 2 years ago

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.

selectdev commented 2 years ago

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.

selectdev commented 2 years ago

This is still awaiting review from @mat-1

stjet commented 2 years ago

@Codemonkey51 Please review! Tested and working.