multi-account-containers
multi-account-containers copied to clipboard
adds an ability to reopen current tab in the specific container with a shortcut
Before submitting your pull request
- [x] I agree to license my code under the MPL 2.0 license.
- [x] I rebased my work on top of the main branch.
- [x] I ran
npm testand all tests passed. - [x] I added test coverages if relevant.
Description
Issue https://github.com/mozilla/multi-account-containers/issues/1792
Type of change
Select all that apply.
- [ ] Bug fix
- [x] New feature
- [ ] Major change (fix or feature that would cause existing functionality to work differently than in the current version)
https://github.com/user-attachments/assets/4291315b-d808-4bcf-92eb-e31511614c25
PR to localization: https://github.com/mozilla-l10n/multi-account-containers-l10n/pull/31
Hey, guys. Can I have a review, please? @dannycolin @groovecoder @rafeerahman
Hey, guys. Can I have a review, please? @dannycolin @groovecoder @rafeerahman
😭😭😭 Can we have this?
Hey, guys. Can I have a review, please? @dannycolin @groovecoder @rafeerahman
😭😭😭 Can we have this?
@lesleyjanenorton
Hey, guys. Can I have a review, please? @dannycolin @groovecoder @rafeerahman
😭😭😭 Can we have this?
@lesleyjanenorton
@antondudakov this is open source and sometimes people just have no time to pay attentions because completely overloaded on their regular job.
What I could advise to increase chances for faster review and acceptance of PR:
- implement only functionality that you would like to get: less code to review - easier to review and bigger chance to be reviewed.
- if you would like to have code refactored - create separate PR: less code to review and with no new functionality - easier to review and bigger chance to be reviewed.
- create a GitHub Issue that will describe what is the feature and how it will work, sometimes discussion in the issue result to completely different direction of feature implementation than initially it was expected.
Hey, guys. Can I have a review, please? @dannycolin @groovecoder @rafeerahman
😭😭😭 Can we have this?
@lesleyjanenorton
@antondudakov this is open source and sometimes people just have no time to pay attentions because completely overloaded on their regular job.
What I could advise to increase chances for faster review and acceptance of PR:
* implement only functionality that you would like to get: less code to review - easier to review and bigger chance to be reviewed. * if you would like to have code refactored - create separate PR: less code to review and with no new functionality - easier to review and bigger chance to be reviewed. * create a GitHub Issue that will describe what is the feature and how it will work, sometimes discussion in the issue result to completely different direction of feature implementation than initially it was expected.
This is exactly what I did.
- The code that implement feature is just around 50 lines of code;
- it's not a refactoring;
- There's an issue https://github.com/mozilla/multi-account-containers/issues/1792, I also mentioned my PR in comments in the issue
Hey @achernyakevich-sc, I hope you're doing well. Could you provide a review, please? I've addressed all your comments. 🙏🥹
@achernyakevich-sc, if you have a spare minute, please review. All addressed
Hey, @bakulf, can you give feedback on this PR as well? I would really appreciate it.