-
-
Notifications
You must be signed in to change notification settings - Fork 280
feat: support refresh for issues list #325
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
feat: support refresh for issues list #325
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @GZLiew, Thank you for working on this 🙌🏻
I wanted to do this differently but that would require some rework on how the TUI is implemented. So, I think your approach is fine for now and I will improve it later.
I do have some comments tho.
- Let's use standard
CTRL + R
/F5
as a shortcut for refreshing. - Could you please extend this to
jira epic list <epic_id>
andjira sprint list <sprint_id>
?
Thank you!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the work 👍🏽 (just another random contributor here)
Could you please update Readme and comments so they stay up to date with latest shortcuts used?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for working on this @GZLiew and thank you for reviewing @adolsalamanca, it helps! 🎉
This PR add refreshing behaviour when the
<C-R>
orF5
key is pressed when viewing issues listrelating to #292 but with no interval implemented