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

Dynamic notices not updating if we use nav bar search to open another table #2204

Closed
mikaalanwar opened this issue Nov 6, 2023 · 1 comment
Labels
status:needs_triage For all issues that need to be processed type:bug An unexpected problem or unintended behavior

Comments

@mikaalanwar
Copy link
Contributor

mikaalanwar commented Nov 6, 2023

Dynamic notices are not updated/changed if we use the "nav bar" search at the top to open another table. However, if we navigate to another page (e.g. search or home page etc.) and then choose another table, the dynamic notices do get updated. In the network tab in Chrome Debugging Tools, it can be observed that an explicit web request is not sent to the Notices API in the former case, however, it is sent in the latter, having a URL similar to the one given below:

http://localhost:5000/api/notices/v0/table?resource=table&key=TABLE_PATH

Expected Behavior

Dynamic notices should be updated whenever we open the table detail page for a different table (regardless of whether we traversed from the home page or went from one table detail page to another using the search box in the navigation bar at the top).

Current Behavior

Dynamic notices are not updated/changed if we use the "nav bar" search at the top to open another table.

Possible Solution

React code is updated in such a way that it triggers the request every time the table detail page is changed (i.e. we go from the table detail page for one table to another).

Steps to Reproduce

  1. Configure dynamic notices (by adding a notice client and enabling it in settings) for Amundsen.
  2. Open the table detail page for any table.
  3. Now navigate to another table detail page using the nav bar search at the top.
  4. You will notice that the dynamic notices are not updated. This can be cross-checked via Chrome debugger.

Screenshots (if appropriate)

N/A

Context

We are unable to use a critical feature that heavily leverages the dynamic notices in Amundsen.

Your Environment

@mikaalanwar mikaalanwar added status:needs_triage For all issues that need to be processed type:bug An unexpected problem or unintended behavior labels Nov 6, 2023
@mikaalanwar mikaalanwar changed the title Dynamic Notices not updating if we use nav bar search to open another table Dynamic notices not updating if we use nav bar search to open another table Nov 6, 2023
mikaalanwar added a commit to deliveryhero/amundsen that referenced this issue Nov 15, 2023
mikaalanwar added a commit to deliveryhero/amundsen that referenced this issue Nov 16, 2023
@mikaalanwar
Copy link
Contributor Author

mikaalanwar commented Nov 20, 2023

Closing this as the issue was fixed by the pull request #2210

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:needs_triage For all issues that need to be processed type:bug An unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

1 participant