Skip to content
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

Not able to edit file-level license #955

Closed
capfei opened this issue Mar 1, 2022 · 6 comments
Closed

Not able to edit file-level license #955

capfei opened this issue Mar 1, 2022 · 6 comments

Comments

@capfei
Copy link
Member

capfei commented Mar 1, 2022

The edit button does nothing when trying to edit the license info for a file. Also, clicking on the license field does nothing.
image

qtomlinson added a commit to qtomlinson/website that referenced this issue Apr 27, 2022
qtomlinson added a commit to qtomlinson/website that referenced this issue May 9, 2022
qtomlinson added a commit to qtomlinson/website that referenced this issue May 24, 2022
@bduranc
Copy link

bduranc commented Jan 9, 2023

This appears to be supported now, but the quality of this functionality is still very poor:

  1. the resulting dialog's radio buttons is confusing - only one radio in the whole form can be selected at any time:

Capture

  1. If you repetitively hit "add group" and create multiple empty groups, you cannot delete them after.

image

@capfei
Copy link
Member Author

capfei commented Jan 9, 2023

Can confirm that I am able to do file-level edits (haven't submitted a PR, though) and experiencing the same issue with the radio buttons.

@qtomlinson
Copy link
Collaborator

qtomlinson commented Jan 10, 2023

Previous commit reused LicensePicker for license editing. It seems that LicensePicker needs some work as well.

@qtomlinson
Copy link
Collaborator

qtomlinson commented Jan 10, 2023

@capfei @bduranc The improvement needed in LicensePicker is more general and not limited to "file-level license" editing. Editing license in the "Described and Licensed" of the definition page shows the same dialogue and needs the same improvement. The original use case of "Not able to edit file-level license" is now supported. The improvement for LicensePicker is better documented in a separate issue, so that all the test cases (file-level and declared license) can be addressed.

image

@bduranc
Copy link

bduranc commented Jan 10, 2023

Alright, I will open new issue to discuss the usability of this dialog box.

It is worth mentioning that in the old UI, this function did work much better because none of these issues existed.

@bduranc
Copy link

bduranc commented Jan 13, 2023

I confirmed I was able to submit this curation PR: clearlydefined/curated-data#23164

I created issue #1013 addresses the larger problemswith the LicensePicker component itself. But for the original scope of this issue, it looks like the problem is fixed.

@capfei do we want to close this?

@capfei capfei closed this as completed Jan 13, 2023
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

No branches or pull requests

3 participants