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

favourite button in frontend not working #82

Closed
siddharthgarishakurthi opened this issue Aug 14, 2019 · 10 comments
Closed

favourite button in frontend not working #82

siddharthgarishakurthi opened this issue Aug 14, 2019 · 10 comments
Labels
keep fresh Disables stalebot from closing an issue

Comments

@siddharthgarishakurthi
Copy link

Even after clicking on the favorite option , i am unable to store it as favorite table.

@jornh
Copy link
Contributor

jornh commented Aug 14, 2019

You need to have user authentication enabled for this to work. It could get clarified by documentation (see some proposed draft WIP in #53)

@ttannis
Copy link
Contributor

ttannis commented Aug 16, 2019

@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?

@dschaller dschaller added the keep fresh Disables stalebot from closing an issue label Aug 17, 2019
@jornh
Copy link
Contributor

jornh commented Sep 6, 2019

@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.

@siddharthgarishakurthi
Copy link
Author

@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.

@jornh
Copy link
Contributor

jornh commented Sep 6, 2019

No worries. OK, got it, so as I understand it the issue is more like:

“when trying to set up auth using the OIDC guide - how do I verify/troubleshoot that I configured each step correctly”?

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
Copy link
Member

feng-tao commented Sep 6, 2019

you mean bookmark button(the star shape)? CC @ttannis @danwom

@danwom
Copy link

danwom commented Sep 6, 2019

@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 indexUsers feature to be enabled). There is an implicit assumption that authentication will be used, but perhaps some instances won't have it (maybe just hide the site behind a VPN?). In those cases we could consider hiding the bookmarks behind a feature flag.

@jornh jornh mentioned this issue Sep 11, 2019
3 tasks
feng-tao pushed a commit that referenced this issue Sep 11, 2019
* bump docker images to latest available on hub

* Fix for #82 unable to favorite by using test user
@jornh
Copy link
Contributor

jornh commented Sep 11, 2019

@siddharthgarishakurthi I made a super-simple fix in PR #97 to the issue you originally posted in #82 (comment) above.

Please do a git pull of this repo - and see if it works for you for at least demoing with the Quickstart setup. (Then we're at least one although tiny step towards an improved Amundsen unboxing experirence).

@siddharthgarishakurthi
Copy link
Author

@jornh thank you for the changes .. The favourite button is working now and i am able to see the My bookmarks section in frontend.

@feng-tao
Copy link
Member

thanks @siddharthgarishakurthi and @jornh

jakemongaya pushed a commit to jakemongaya/amundsen that referenced this issue Dec 11, 2019
* [AMD-149] Add updated timestamp into node and relation

* Update
gabrielucelli pushed a commit to gabrielucelli/amundsen that referenced this issue Jan 28, 2020
benrifkind pushed a commit to benrifkind/amundsen that referenced this issue Dec 16, 2020
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
* bump docker images to latest available on hub

* Fix for amundsen-io#82 unable to favorite by using test user
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
* [AMD-149] Add updated timestamp into node and relation

* Update
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
dorianj pushed a commit to dorianj/amundsen that referenced this issue Apr 25, 2021
* [AMD-1157] Boost exact match of display_name on table

* Update
feng-tao pushed a commit that referenced this issue May 7, 2021
* [AMD-149] Add updated timestamp into node and relation

* Update
feng-tao pushed a commit that referenced this issue May 7, 2021
* bump docker images to latest available on hub

* Fix for #82 unable to favorite by using test user
feng-tao added a commit that referenced this issue May 7, 2021
feng-tao pushed a commit that referenced this issue May 7, 2021
* [AMD-1157] Boost exact match of display_name on table

* Update
feng-tao pushed a commit that referenced this issue May 7, 2021
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
zacr pushed a commit to SaltIO/amundsen that referenced this issue May 13, 2022
zacr pushed a commit to SaltIO/amundsen that referenced this issue May 13, 2022
* [AMD-1157] Boost exact match of display_name on table

* Update
hansadriaans pushed a commit to DataChefHQ/amundsen that referenced this issue Jun 30, 2022
Signed-off-by: Allison Suarez Miranda <asuarezmiranda@lyft.com>
hansadriaans pushed a commit to DataChefHQ/amundsen that referenced this issue Jun 30, 2022
* [AMD-149] Add updated timestamp into node and relation

* Update
hansadriaans pushed a commit to DataChefHQ/amundsen that referenced this issue Jun 30, 2022
hansadriaans pushed a commit to DataChefHQ/amundsen that referenced this issue Jun 30, 2022
hansadriaans pushed a commit to DataChefHQ/amundsen that referenced this issue Jun 30, 2022
* [AMD-1157] Boost exact match of display_name on table

* Update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keep fresh Disables stalebot from closing an issue
Projects
None yet
Development

No branches or pull requests

6 participants