-
Notifications
You must be signed in to change notification settings - Fork 967
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
favourite button in frontend not working #82
Comments
You need to have user authentication enabled for this to work. It could get clarified by documentation (see some proposed draft WIP in #53) |
@siddharthgarishakurthi second to what @jornh said, I think it is likely that the feature is not configured correctly, therefore the request fails, and therefore the UI does not update. When you open your network tab in the browser and click the favorite button, are you getting an error? |
@siddharthgarishakurthi did you get this working or not? As it stands I don't see any actionable info for the project. If you still think there's an issue please provide additional info (e.g. steps to reproduce) so someone can try to help resolve it. |
@jornh it is not working for me Can you please share how to enable the user authentication if possible as i could not understand the doc provided in this link : https://github.com/lyft/amundsen/blob/master/docs/authentication/oidc.md Sorry for late reply .. I was busy with some other projects. |
No worries. OK, got it, so as I understand it the issue is more like:
Personally I haven’t gotten to try setting up OIDC. So I can’t give you a direct answer. I have merely set up one hardcoded userid to try out all the user profile stuff. What I do know is that ING has it working in their setup (they built it) - and so far I know of one thread on Slack that sadly died, til now: https://amundsenworkspace.slack.com/archives/CGFBVT23V/p1567144954013700?thread_ts=1567121193.012100&channel=CGFBVT23V&message_ts=1567144954.013700 How far have you gotten compared to that thread? - can you describe steps you took and results/error messages you got - or specific questions you have? Maybe point out specific places in the guide that leave you in doubt. If you can do that I’m sure we can collective get to a better situation. |
@feng-tao The bookmark/favorite/follow star is not hidden by any config at the moment. At minimum the bookmark feature requires authentication the bookmarks can be linked to the user. (This does not require the |
* bump docker images to latest available on hub * Fix for #82 unable to favorite by using test user
@siddharthgarishakurthi I made a super-simple fix in PR #97 to the issue you originally posted in #82 (comment) above. Please do a |
@jornh thank you for the changes .. The favourite button is working now and i am able to see the My bookmarks section in frontend. |
thanks @siddharthgarishakurthi and @jornh |
* [AMD-149] Add updated timestamp into node and relation * Update
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
* bump docker images to latest available on hub * Fix for amundsen-io#82 unable to favorite by using test user
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
* [AMD-149] Add updated timestamp into node and relation * Update
* [AMD-1157] Boost exact match of display_name on table * Update
* [AMD-149] Add updated timestamp into node and relation * Update
* bump docker images to latest available on hub * Fix for #82 unable to favorite by using test user
* [AMD-1157] Boost exact match of display_name on table * Update
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
* [AMD-1157] Boost exact match of display_name on table * Update
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
* [AMD-149] Add updated timestamp into node and relation * Update
* [AMD-1157] Boost exact match of display_name on table * Update
Even after clicking on the favorite option , i am unable to store it as favorite table.
The text was updated successfully, but these errors were encountered: