-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
one-click-review-submission
- Restore feature
#7015
one-click-review-submission
- Restore feature
#7015
Conversation
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.
Thank you for the PR! Unfortunately I think this will break GitHub Enterprise. The solution just be to use ['.form-actions', '.Overlay-footer']
.
The array style will allow for comments to specify that the first one only exists for compat
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.
If this works I'll merge it today and release a new version
one-click-review-submission
- Fix selectorone-click-review-submission
- Restore feature
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.
Confirmed this is still working on GHE 3.9.4
Thank you @tanmayrajani and @busches! |
Thanks! Side question: do you know how long it usually takes for a new release to make it to the Chrome extensions store? Days? Weeks? |
Usually minutes, but occasionally hours. Every year or 2 years it's blocked for further review. If you don't see it tomorrow we can ping Sindre |
It's out and works for me after clearing cache in extension settings :) |
Fixes #7010
.parentElement
instead of.closest('.form-actions')
since it looks like there's no.form-actions
anymoreTest URLs
Any open PR URLs where you have access to review the changes
e.g. #7005
Screenshot
with tooltip