Skip to content

Comments

Added request access popup to file pg [ref #4897]#5956

Merged
kcondon merged 3 commits intodevelopfrom
4897-file-pg-request-access-fix
Jun 25, 2019
Merged

Added request access popup to file pg [ref #4897]#5956
kcondon merged 3 commits intodevelopfrom
4897-file-pg-request-access-fix

Conversation

@mheppler
Copy link
Contributor

Related Issues

Pull Request Checklist

@coveralls
Copy link

coveralls commented Jun 19, 2019

Coverage Status

Coverage remained the same at 19.676% when pulling ab03bb2 on 4897-file-pg-request-access-fix into 3dde216 on develop.

@landreev landreev removed their assignment Jun 25, 2019
@kcondon
Copy link
Contributor

kcondon commented Jun 25, 2019

@mheppler Request Access popup no longer sends request to admin from either dataset or file landing page. Works in develop. See dvn-build for develop, internal for your branch

@mheppler
Copy link
Contributor Author

mheppler commented Jun 25, 2019

Removed a snippet of code from the file pg that seemed to have been an unintentional rogue pasting of ui:param components outside of the dialogs they were intended to be included in.

Unfortunately, these rogue lines of code did not change the behavior of the Request Access btns for me in my localhost. The buttons on the file pg and dataset pg all worked as expected, sending notifications to the dataverse admin through the system (no email server is setup for my localhost, so the server log complains about that).

Taking a look at the behavior of this branch on dataverse-internal, I noticed the Request Access btn works as expected on the file pg (see attachments), and intermittently on the dataset pg.

Screen Shot 2019-06-25 at 1 48 26 PM

Screen Shot 2019-06-25 at 1 48 37 PM

Screen Shot 2019-06-25 at 1 54 47 PM

While I am not sure if cleaning up the rogue code snippet changes anything, maybe the rogue popup btns on the dataset pg, reported in #5958 have something to do with the intermittent behavior of the Request Access btn.

@kcondon -- happy to update this branch with develop once that gets merge, if you want to put off testing this again.

@mheppler mheppler removed their assignment Jun 25, 2019
@kcondon
Copy link
Contributor

kcondon commented Jun 25, 2019

@mheppler does request access button change state to access requested and the actual username appear as a request under the files permissions?

@mheppler
Copy link
Contributor Author

@kcondon Yes. See attached screenshots showing both.

@kcondon
Copy link
Contributor

kcondon commented Jun 25, 2019

@mheppler was able to reproduce it: it happens when extra accept/cancel buttons appear at bottom of dataset due to this bug: #5958 So, let's fix/merge that one before retesting and merging this one.

@mheppler
Copy link
Contributor Author

Merged the latest and greatest from develop (including #5958 dataset pg btn bug fixes), and pushed those changes to this branch. Confirmed the Request Access btn function on datasets that previously had the rogue Accept and Cancel btns at the bottom of the pg.

@kcondon kcondon merged commit 8643379 into develop Jun 25, 2019
@kcondon kcondon deleted the 4897-file-pg-request-access-fix branch June 25, 2019 21:24
@kcondon kcondon self-assigned this Jun 25, 2019
@djbrooke djbrooke added this to the 4.15.1 milestone Jul 10, 2019
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.

5 participants