Skip to content
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

Strip channel Guild/Channel ID from minId/maxId input #156

Closed
wants to merge 2 commits into from
Closed

Strip channel Guild/Channel ID from minId/maxId input #156

wants to merge 2 commits into from

Conversation

smatchcube
Copy link

@smatchcube smatchcube commented Dec 24, 2020

Hi,

Copying the message ID from a message (while hovering a message with shift key pressed) copy both the channel ID and the message ID (separated with a '-' character) but the input boxes "After message with ID" and "Before message with ID" take only the user ID.
My little changes allow the user to past directly the input as directly copied to the input boxes without needing to manually strip the channel ID.
I also updated the documentation which was outdated.

Thanks for this useful tool!


This change is Reviewable

@victornpb
Copy link
Owner

I don’t understand, isn't it easier to just right click instead of shift-right click?

@smatchcube
Copy link
Author

Oh I see, the "right click" -> "Copy ID" action is only available with developer mode enabled. My change would allow the function to work even without having developer mode enabled but it's not that important. Feel free to close the issue if you don't want to add this functionality.

@victornpb
Copy link
Owner

victornpb commented May 2, 2022

I just never knew this existed😅, because I had developer mode enabled since forever (well I'm a developer😂).

In the last version I added an easier way of selecting messages by just clicking on them. So in most circumstances ou shouldn't need to manually input IDs.

If you still think this would be necessary, please lmk

@smatchcube
Copy link
Author

The new way of selecting messages perfectly solves my problem and make it easier for new users (no need to enable developer mode anymore!).
Thank you!

@smatchcube smatchcube closed this May 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants