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

✨ Add global Jira link #537

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

C-Valen
Copy link
Member

@C-Valen C-Valen commented Feb 12, 2025

OSIDB-3964 Add global Jira link

Checklist:

  • Commits consolidated
  • Changelog updated
  • Test cases added/updated
  • Jira ticket updated

Summary:

Adds a link to Jira task if existing in the top right side of the flaw form view, just below the BZ link.
This requeriment appeared cause previously the only Jira link was on the internal comments, making it for read-only users who don't have an Jira API key set, difficult to quickly access to the related Jira task of a flaw.

Changes:

  • Adds the new Jira link on the flaw form view

Screenshot:

image

Closes OSIDB-3964

@C-Valen C-Valen added the enhancement New feature or request label Feb 12, 2025
@C-Valen C-Valen requested a review from a team February 12, 2025 09:00
@C-Valen C-Valen self-assigned this Feb 12, 2025
@C-Valen C-Valen force-pushed the feature/OSIDB-3964-global-jira-link branch from 10a441b to 4ae377e Compare February 12, 2025 09:33
@@ -175,15 +176,24 @@ const createdDate = computed(() => {
>
<div class="row osim-flaw-form-section pt-0">
<div class="osim-flaw-form-header">
<div v-if="flaw.meta_attr?.bz_id" class="osim-flaw-header-link">
<div v-if="flaw.meta_attr?.bz_id || flaw.task_key" class="osim-flaw-header-link">
<a
Copy link
Member

Choose a reason for hiding this comment

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

maybe add another v-if here, since the parent div can be displayed anyway

@@ -523,3 +563,6 @@ describe('flawForm', () => {
expect(wrapper.emitted()).toHaveProperty('refresh:flaw');
});
});
function importOriginal(arg0: string) {
Copy link
Member

Choose a reason for hiding this comment

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

This is not used

@@ -523,3 +563,6 @@ describe('flawForm', () => {
expect(wrapper.emitted()).toHaveProperty('refresh:flaw');
});
});
function importOriginal(arg0: string) {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can we remove this?

@C-Valen C-Valen force-pushed the feature/OSIDB-3964-global-jira-link branch from 4ae377e to eccf240 Compare February 12, 2025 15:13
@C-Valen C-Valen force-pushed the feature/OSIDB-3964-global-jira-link branch from eccf240 to 726d2ab Compare February 12, 2025 15:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants