Skip to content

AXON-771 - added loading indicator for Jira issue auto-refreshes #695

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

AXON-771 - added loading indicator for Jira issue auto-refreshes #695

wants to merge 5 commits into from

Conversation

sky-ocean-spirits
Copy link
Collaborator

@sky-ocean-spirits sky-ocean-spirits commented Aug 6, 2025 •
edited
Loading

What Is This Change?

Issues (N608, 771):
When the Jira issue view is auto-refreshing (switching between VS Code tabs, the editor's window focus), the refresh icon is not spinning

Solution:
Replaced direct setState for the Refresh button with message-based loading state management in forceUpdateIssue method

Recordings:
Before:
https://www.loom.com/share/9f1f70e2329f4e30bb0300c79c1d68eb

After:
https://www.loom.com/share/39176120a2354fe094c82a0d9ea3bbbf

How Has This Been Tested?

Basic checks:

  • npm run lint
  • npm run test

Advanced checks:

  • If Atlassian employee & Bitbucket changes: did you test with DC in mind? See Instructions

Recommendations:

  • Update the CHANGELOG if making a user facing change

@sky-ocean-spirits sky-ocean-spirits force-pushed the AXON-771-visual-indication-for-auto-refresh-jira-issue branch from 0a157c3 to 39ec8bd Compare August 8, 2025 08:03
BHulovatyi
BHulovatyi previously approved these changes Aug 8, 2025
View reviewed changes
Copy link
Collaborator

@BHulovatyi BHulovatyi left a comment

Choose a reason for hiding this comment

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

Looks good!

sspanchenko
sspanchenko previously approved these changes Aug 8, 2025
View reviewed changes
@sky-ocean-spirits sky-ocean-spirits marked this pull request as ready for review August 8, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bwieger-atlassian-com bwieger-atlassian-com Awaiting requested review from bwieger-atlassian-com bwieger-atlassian-com is a code owner

@sdzh-atlassian sdzh-atlassian Awaiting requested review from sdzh-atlassian sdzh-atlassian is a code owner

@cabella-dot cabella-dot Awaiting requested review from cabella-dot cabella-dot is a code owner

@marcomura marcomura Awaiting requested review from marcomura marcomura is a code owner

@akalia-atlassian akalia-atlassian Awaiting requested review from akalia-atlassian akalia-atlassian is a code owner

@pastelsky pastelsky Awaiting requested review from pastelsky pastelsky is a code owner

@bhorai-atl bhorai-atl Awaiting requested review from bhorai-atl bhorai-atl is a code owner

@BHulovatyi BHulovatyi Awaiting requested review from BHulovatyi

@sspanchenko sspanchenko Awaiting requested review from sspanchenko

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sky-ocean-spirits @sspanchenko @BHulovatyi