Skip to content

fix: added fix for changelog refetching multiple times on tab change #8545

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

Merged
merged 1 commit into from
Jul 16, 2025

Conversation

ahrefabhi
Copy link
Contributor

@ahrefabhi ahrefabhi commented Jul 16, 2025

πŸ“„ Summary


βœ… Changes

  • Fix for changelog refetching multiple times on tab switch

Important

Fixes multiple refetches of changelog on tab switch in AppLayout by modifying useEffect dependency array to only include isVisible.

  • Behavior:
    • Fixes multiple refetches of changelog on tab switch in AppLayout by modifying useEffect dependency array to only include isVisible.

This description was created by Ellipsis for 33fd5d6. You can customize this summary. It will automatically update as commits are pushed.

@ahrefabhi ahrefabhi requested review from YounixM and a team as code owners July 16, 2025 10:56
@github-actions github-actions bot added the bug Something isn't working label Jul 16, 2025
Copy link
Contributor

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Changes requested ❌

Reviewed everything up to 33fd5d6 in 1 minute and 36 seconds. Click for details.
  • Reviewed 14 lines of code in 1 files
  • Skipped 0 files when reviewing.
  • Skipped posting 0 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with πŸ‘ or πŸ‘Ž to teach Ellipsis.

Workflow ID: wflow_U297zW5eS6Ora7mm

You can customize Ellipsis by changing your verbosity settings, reacting with πŸ‘ or πŸ‘Ž, replying to comments, or adding code review rules.

@YounixM YounixM merged commit 7ce1a1c into main Jul 16, 2025
14 checks passed
@YounixM YounixM deleted the chore/changelog-refetch-fix branch July 16, 2025 11:05
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

Successfully merging this pull request may close these issues.

2 participants