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

Axon 137 sidebar UI new sidebar view for jq ls #154

Merged
merged 14 commits into from
Mar 4, 2025

Conversation

cabella-dot
Copy link
Contributor

@cabella-dot cabella-dot commented Feb 26, 2025

  • create new Custom Jql vew provider
  • add searchIssueHelper to hold searchable issues
  • Next:
  • create status tags for issue node
  • enable custom jql view in config page

Demo:
https://www.loom.com/share/6cd9a9ec8fd14dbdbe7aa78966ab4b69

@cabella-dot cabella-dot marked this pull request as ready for review February 28, 2025 21:39
marcomura
marcomura previously approved these changes Mar 3, 2025
Copy link
Contributor

@marcomura marcomura 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! thanks!

marcomura
marcomura previously approved these changes Mar 4, 2025
@cabella-dot cabella-dot force-pushed the AXON-137-sidebar-ui-new-sidebar-view-for-jq-ls branch from 97904c9 to 24f4c60 Compare March 4, 2025 01:23
marcomura
marcomura previously approved these changes Mar 4, 2025
sdzh-atlassian
sdzh-atlassian previously approved these changes Mar 4, 2025
Copy link
Member

@sdzh-atlassian sdzh-atlassian left a comment

Choose a reason for hiding this comment

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

Nice! Good to see some tests going into place, too! 🚢

Copy link
Collaborator

@bwieger-atlassian-com bwieger-atlassian-com left a comment

Choose a reason for hiding this comment

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

  • Enhancement / Bug: If I log out of all sites, I see JQL filters with no results. Instead I should see a call to action to sign in.
  • Bug: After logging out, and then logging back in, JQL filters go entirely blank.

@cabella-dot cabella-dot force-pushed the AXON-137-sidebar-ui-new-sidebar-view-for-jq-ls branch from 24f4c60 to eab13a7 Compare March 4, 2025 22:14
@cabella-dot
Copy link
Contributor Author

@bwieger-atlassian-com
Copy link
Collaborator

@cabella-dot Reviewed! LGTM

Here's my review video: https://www.loom.com/share/7045725bcdfd4ffc8b5e78cbf7d677f6?sid=bb5ed99c-8a20-418d-9e5a-cd654025e69c

It looks like you resolved one of the JQL bugs with this PR. Good work! 🚀

There is still an outstanding pre-existing bug. But not an expectation to solve in this PR.

@cabella-dot cabella-dot merged commit 7a6cfbf into main Mar 4, 2025
2 checks passed
@cabella-dot cabella-dot deleted the AXON-137-sidebar-ui-new-sidebar-view-for-jq-ls branch March 4, 2025 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants