-
Notifications
You must be signed in to change notification settings - Fork 117
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
VPN-5707 Remove the "Messages - REMOVE ME" option from the Developer menu #8270
Conversation
@@ -1,80 +0,0 @@ | |||
/* This Source Code Form is subject to the terms of the Mozilla Public |
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.
Is this not useful though? Why did that menu option even have the REMOVE ME
in it?
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.
@MattLichtenstein Looks like this was added by you September of last year. Can you confirm that this isn't needed?
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.
this was actually a Baku thing - and I don't ever use this so delete away!
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.
My gut reaction was that we should keep the menu, but I just opened it for the first time and it looks like it just displays the same messages that are in the inbox. I may be missing some functionality, but if that's all it is, I can't think of how it would be useful. If there is some additional functionality, I agree we should keep it.
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.
this view preceeded the message inbox UI. it was a way to test message addons (and before that, all addons) before we had proper UI
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.
👍🏻
Description
Santiago said he hates it and we must keep Santiago happy so this has got to go. Sorry I don't make the rules! 😁
Reference
Checklist