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

Remove standard license header field in created issue for new license submission #452

Closed
jlovejoy opened this issue Mar 29, 2023 · 13 comments
Assignees
Labels
Submit New License Issues related to the submit new license feature

Comments

@jlovejoy
Copy link
Member

when we updated the submission form fields, we did not also update the display to no longer show
the Standard License Header field even tho that is no longer asked in the submission form

It might be nice to add a link to the corresponding Github issue

@BassCoder2808
Copy link
Contributor

Hi @jlovejoy, I would like to work on this issue, can you please help me with how can I get started with it and what all fields need to be updated?

@jlovejoy
Copy link
Member Author

@BassCoder2808 the submission form does not ask for a Standard License Header any longer, but once submitted, the license request page still includes the Standard License Header field, see for example https://tools.spdx.org/app/license_requests/213/

We need to remove the Standard License Header field from the license request template.

does that make sense?

@jlovejoy jlovejoy added the Submit New License Issues related to the submit new license feature label Mar 31, 2023
@BassCoder2808
Copy link
Contributor

BassCoder2808 commented Mar 31, 2023

Ok yes, I have understood the issue now, so now instead of that, we will put the github issue link.
Hi @jlovejoy I have updated my PR please let me know if anything needs to be changed in that

@BanulaKumarage
Copy link
Contributor

@BassCoder2808 the submission form does not ask for a Standard License Header any longer, but once submitted, the license request page still includes the Standard License Header field, see for example https://tools.spdx.org/app/license_requests/213/

We need to remove the Standard License Header field from the license request template.

does that make sense?

@jlovejoy was there a specific purpose for putting the standard license header earlier on the license information page? Was it asked from the user earlier?

@jlovejoy
Copy link
Member Author

jlovejoy commented Apr 1, 2023

@BanulaKumarage yes, it used to be asked for in the submission form, but it is not something that is used in very many licenses, and in the unlikely case that it is used, someone from SPDX-legal needs to really look at it and probably code it in the XML manually. So we decided it wasn't necessary to ask and removed from the submission form

@BanulaKumarage
Copy link
Contributor

@BanulaKumarage yes, it used to be asked for in the submission form, but it is not something that is used in very many licenses, and in the unlikely case that it is used, someone from SPDX-legal needs to really look at it and probably code it in the XML manually. So we decided it wasn't necessary to ask and removed from the submission form

Thank you @jlovejoy I got the idea.

@jlovejoy jlovejoy changed the title license request page updates Remove standard license header field in created issue for new license submission Jun 21, 2023
@jlovejoy
Copy link
Member Author

@BassCoder2808 - has this been solved/merged? looks like there may be 2 PRs?

@BassCoder2808
Copy link
Contributor

Hi @jlovejoy actually there has been some miscommunications like there are 2 issues and 2 PRs that solve the 2 issues separately which in the end intend to do the same task.
So what I feel is that we can move forward with this one (just because it was opened earlier and solves one more issue)
and the PR to move forward with #456

@goneall
Copy link
Member

goneall commented Jun 24, 2023

@BassCoder2808 - I merged in PR #450 which seems to have caused a merge conflict for #456 - If you could resolve and address the comments from @rtgdk I'll merge it in for the next release.

@jlovejoy
Copy link
Member Author

the standard license header field is still showing up in the list of submitted licenses in the online, see https://tools.spdx.org/app/license_requests/251/

but it is not listed in the Github issue spdx/license-list-XML#2033

@BassCoder2808
Copy link
Contributor

Hi @jlovejoy yes I have checked the code even though the previous PR is not merged till now, but once that get's merged I will make the changes, so the problem is in license_information.html wherein we have not removed the standard license header field even though in code we have removed it

@goneall
Copy link
Member

goneall commented Jan 19, 2024

@BassCoder2808 @jlovejoy - Checking to see if this is resolved or if it should be left open

@BassCoder2808
Copy link
Contributor

This issue has been solved @goneall as far as I remember

@goneall goneall closed this as completed Jan 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Submit New License Issues related to the submit new license feature
Projects
None yet
Development

No branches or pull requests

4 participants