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 showing incorrect value(s) #2205

Closed
mikaalanwar opened this issue Nov 6, 2023 · 1 comment
Closed

Dynamic notices showing incorrect value(s) #2205

mikaalanwar opened this issue Nov 6, 2023 · 1 comment
Labels
status:completed Issue is completed and on master type:bug An unexpected problem or unintended behavior

Comments

@mikaalanwar
Copy link
Contributor

Dynamic notices showing stale values from other tables and/or not updating correctly. We have a custom notice client that sends a request to an external system and fetches a value, based on which an HTML message/link is generated to be displayed as a dynamic notice. In our scenario, the external system can be quite slow at times, resulting in a response time of around 5-7 seconds at times. It has been observed that Dynamic Notices displayed at times don't even get updated or in some cases, they do get updated but for the initial 4-5 seconds they displayed the stale/older value (which is incorrect for the table against which it's displayed).

Expected Behavior

Either the dynamic notices are not displayed until a newer/proper value is loaded or it doesn't render altogether/remains hidden until a proper/newer value has been returned by the notice client.

Current Behavior

In some cases, the stale/older/wrong value is displayed by the notice client (corresponding to the previously opened table) while in other cases, the correct value is displayed after a delay with the stale/incorrect value being displayed for the initial 4-5 seconds (corresponding to the latency of the Notice Client)

Possible Solution

The React code is updated in such a way that dynamic notices for any given page are only displayed after a proper response has been received from the notice client, rather than displaying the stale/incorrect value.

Steps to Reproduce

  1. Create a Notice Client (that simulates a slow web request e.g. returning a dynamic notice response in 5 seconds)
  2. Open a given table (a proper dynamic notice will be displayed)
  3. Navigate to another table and you will observe that the old/stale value is still displayed until the newer one is loaded while in some cases the value of the dynamic notice is not updated at all.

Screenshots (if appropriate)

N/A

Context

This issue prevents us from using a critical feature that heavily involves the use of dynamic notices.

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

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

@kristenarmes kristenarmes added status:completed Issue is completed and on master and removed status:needs_triage For all issues that need to be processed labels Aug 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:completed Issue is completed and on master type:bug An unexpected problem or unintended behavior
Projects
None yet
Development

No branches or pull requests

2 participants