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
Channel that includes messages that have been bookmarked a certain number of times #492
Comments
@nikitarevenco Thanks for creating this issue. I have some thoughts, but let's discuss with the team first. @TheOdinProject/moderators @TheOdinProject/maintainers what do y'all think? |
I'm a bit out of touch with the channels so take my opinion with a grain of salt 😆 But I'm not sure if it'd be worth adding more bloat to the hefty amount of channels we already have. Another issue would be the number of bookmarks that are required, as it's difficult to really decide what the "right" number is. Is 10 bookmarks enough? Would 20 be too many based on the average amount of bookmarks a single post gets? Finally, what people find useful to bookmark can vary wildly, and such a channel could become extremely bloated depending how often the bookmark feature is being used. At a certain point it could become difficult to sift through all of the bookmarks to see what the person finds useful. That said, I also wouldn't be opposed to this feature. Mainly just want to mention some potential issues, but if overall the benefit is significant and outweighs any downside I wouldn't block it. |
From my own experiences across the channels, even some of the most helpful messages don't get a great deal of bookmarks. Generally stuff that gets bookmarked are things sent during 1:1 help and so often only get 1 or 2 bookmarks. Even the few times something might get something like 3-6 bookmarks, it's actually quite often something that's already been bookmarked several times a previous time it was posted, just by different people (or people like me with terrible memory and bookmark anyway just in case). So I'm not so sure that a feature like this would be the most feasible/valuable because:
|
I think 3 bookmarks is the perfect threshold for something like this. Reasoning:
That makes sense, but no single person is going to browse through the entire channel. Maybe people are at most going to browse through the most recent 15-30 posts. It is therefore unlikely that there would be many messages in that range that repeat To further make the channel more searchable, every post can have whatever channel the message originated from. For example, if a message originates from React-help channel, then the bot will mention that, which will let people specifically search for "#react-help" in this bookmark channel to filter only the content they'd want to see |
Complete the following REQUIRED checkboxes:
I have thoroughly read and understand The Odin Project Contributing Guide
The title of this issue follows the
command name: brief description of request
format, e.g./help: add optional @user parameter
The following checkbox is OPTIONAL:
1. Description of the Feature Request:
When I bookmark a message with 📖 I do it because I want to refer to this information at a later date. However, I think it would also be very useful to have a list of useful/helpful messages that other people found helpful. So for example, if some message receives 3x 📖 it probably means that this message has some value / importance to several people and this message could be included in this new channel. let me know what you think about this!
All of this will be automatic with the odin bot.
2. Acceptance Criteria:
bookmarked
(we can decide on a better name)3
(could be less could be more) book emojis.#bookmarked
channel in the same exact format as it sends the message to the user (So it'll include ajump to message
for example)3. Additional Information:
Discord name: Revenco
I've made this suggestion in the
#discord-suggestion
channel and was redirected here. (link to suggestion)I'm happy to work on this feature, although I think a channel in the discord would need to be created for me to be able to test it
The text was updated successfully, but these errors were encountered: