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

[TOS-992 / 989] fix: Do not Allow deletion of uploads when used in Test Plans and add Test Plan Name to popup. #289

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

Raksha-testsigma
Copy link
Contributor

@Raksha-testsigma Raksha-testsigma commented Feb 2, 2023

Jira Tickets
https://testsigma.atlassian.net/browse/TOS-989
https://testsigma.atlassian.net/browse/TOS-992

Description

Fixed the bulk upload deletion while used in Test Plans and Test Cases.[Child Tickets of TOS 782]
Fix does not allow the deletion of an upload used in a Test Plan as a Test Machine.
Added a linked test plans pop-up along with the TEST PLAN NAME with its corresponding TEST MACHINE.

},
error=>{console.log(error)}
)
}
Copy link
Contributor

Choose a reason for hiding this comment

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

@Raksha-testsigma Just correct the indentation of the brackets

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

}
}
}
private openLinkedTestCasesDialog(list) {
Copy link
Contributor

Choose a reason for hiding this comment

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

@Raksha-testsigma Needs a little formatting. In some cases, there is no line gap between two functions. In some other cases, there are quite a few. Use uniform line gaps

<div
*cdkVirtualFor="let linkedEntity of modalData.linkedEntityList ; let index=index; let first=first"
[class.border-separator-t-1]="first"
class="list-view md-pm green-highlight">
<div (click)="openLinkedEntity(linkedEntity.testPlanId)" class="ts-col-100 pointer d-flex" target="_blank">
<div class="text-truncate">

<div class="text-truncate w-235">
Copy link
Contributor

Choose a reason for hiding this comment

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

@Raksha-testsigma Same as in previous comment. Why the extra line gap here?

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.

None yet

2 participants