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

🐛 announcements: The displayed announcement is incorrect #2413

Open
2 tasks done
kurtaking opened this issue Dec 27, 2024 · 0 comments
Open
2 tasks done

🐛 announcements: The displayed announcement is incorrect #2413

kurtaking opened this issue Dec 27, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@kurtaking
Copy link
Contributor

Plugin Name

announcements

📜 Description

This issue is a 1:1 copy from procore-oss/backstage-plugin-announcements#539


The bug occurs when the user is on a particular announcement page and then tries to access another announcement from the search dialogue in the sidebar.

👍 Expected behavior

The second announcement should be rendered.

👎 Actual Behavior with Screenshots

The URL changes to include the ID of the second announcement, but the first one is still visible on the page.

👟 Reproduction steps

Prerequisites:

  1. There is a search button in the sidebar. Clicking on it allows users to search in the dialogue box.
image

Steps to reproduce:

  1. Go to any announcement page.
image 2. Open the search dialogue, and click on another announcement. image 3. The ID of the selected announcement is in the URL, but the first announcement is still visible on the page. image

📃 Provide the context for the Bug.

It seems to me that adding an announcement ID to the useAsync dependency array in the AnnouncementPage component, might solve this problem. Now the component only fetches the announcement once and not when the ID changes.

👀 Have you spent some time to check if this bug has been raised before?

  • I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

Are you willing to submit PR?

No, but I'm happy to collaborate on a PR with someone else

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant