-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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
make access request open new tab #5859
make access request open new tab #5859
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.
LGTM
c41144a
to
20fe7e0
Compare
20fe7e0
to
5a4f201
Compare
48a8787
to
bcdb849
Compare
Codecov Report
@@ Coverage Diff @@
## master #5859 +/- ##
===========================================
- Coverage 77.99% 63.73% -14.26%
===========================================
Files 46 374 +328
Lines 9312 23325 +14013
Branches 0 2607 +2607
===========================================
+ Hits 7263 14867 +7604
- Misses 2049 8445 +6396
- Partials 0 13 +13
Continue to review full report at Codecov.
|
* make access request open new tab * trigger build (cherry picked from commit 01212c3)
* make access request open new tab * trigger build
* make access request open new tab * trigger build
This PR makes request access links open a new tab, as opposed to changing the page from Superset.
@graceguo-supercat @michellethomas