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

fix(unexpected-token): Changed name of key #199

Closed
wants to merge 1 commit into from

Conversation

pret3nti0u5
Copy link
Contributor

Description

Changed key of setLocalStorage() in getUserSelf() from res.default_group to res.name as key default_group is no longer included in the response from endpoint /api/v1/users/self. Changing to res.name fixed the issue.

Changes

Changed key from res.default_group to res.name

How to test

Login using FossologyUI

Closes #198

@GMishx
Copy link
Member

GMishx commented Apr 6, 2022

@pret3nti0u5 name is not the correct field. It should be default_group, just that FOSSology do not expose it.

It was proposed in API v2 here: Shruti3004/FOSSology-REST-API#8

Can you create a PR from it to FOSSology? We can merge it there to make v1 work.

@GMishx GMishx added the wontfix This will not be worked on label Apr 6, 2022
@pret3nti0u5
Copy link
Contributor Author

@GMishx I'll get on that, do you want me to close the PR here?

@GMishx
Copy link
Member

GMishx commented Apr 6, 2022

That will make sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unexpected token error on login attempt in FossologyUI
2 participants