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

Test run - Cannot change test case status on TC view without leaving a comment #1261

Closed
Prome88 opened this issue Dec 10, 2019 · 1 comment
Closed
Assignees
Labels
bug

Comments

@Prome88
Copy link
Contributor

@Prome88 Prome88 commented Dec 10, 2019

Description of problem

Version or commit hash (if applicable)

7.2 @ public tenant

Steps to Reproduce

  1. Create test run
  2. Expand test case
  3. Click on one of status buttons

Actual results

A)
image

B) if users starts clicking on statuses the test run bar goes berserk untill the page is reloaded
image

Expected results

Comment should not be a mandatory field.

Additional info (Python traceback, logs, etc.)

Changing the status via drop-down works but 99% of our users change the status via expanding the TC view.

@asankov

This comment has been minimized.

Copy link
Member

@asankov asankov commented Dec 10, 2019

this should be resolved when #1258 is merged

asankov added a commit to asankov/Kiwi that referenced this issue Dec 11, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring
asankov added a commit to asankov/Kiwi that referenced this issue Dec 11, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring
asankov added a commit to asankov/Kiwi that referenced this issue Dec 11, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring
asankov added a commit to asankov/Kiwi that referenced this issue Dec 12, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring
asankov added a commit to asankov/Kiwi that referenced this issue Dec 12, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`
asankov added a commit to asankov/Kiwi that referenced this issue Dec 12, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`
asankov added a commit to asankov/Kiwi that referenced this issue Dec 12, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`
asankov added a commit to asankov/Kiwi that referenced this issue Dec 14, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses
asankov added a commit to asankov/Kiwi that referenced this issue Dec 20, 2019
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses
@atodorov atodorov added this to the patternfly-migration milestone Jan 17, 2020
asankov added a commit to asankov/Kiwi that referenced this issue Jan 25, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 25, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 25, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 25, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 26, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 28, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 28, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 29, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
asankov added a commit to asankov/Kiwi that referenced this issue Jan 30, 2020
Refs kiwitcms#236
Closes kiwitcms#1261

Clean-up some of the java script in `testrun_actions.js`
Put up some TODO for future refactoring

Use the API to update the execution status

in favour of `updateRunStatus` function

Delete global variable `Nitrate.TestRuns.CaseRunStatus`

Delete `UpdateCaseRunStatusView`

because we will be using the API for updating execution statuses

Disable the button for the current execution status
,because we don't want to update the status of the execution, with the
same status

Also, add `opacity: 0.25` style to the disabled button, so that it is
visible to the user, that the button is disabled
@atodorov atodorov closed this in 27c9857 Jan 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants
You can’t perform that action at this time.