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

Threat numbers are dependent on editing unsaved threats. #889

Closed
mohamedselbohy opened this issue Feb 28, 2024 · 3 comments · Fixed by #910
Closed

Threat numbers are dependent on editing unsaved threats. #889

mohamedselbohy opened this issue Feb 28, 2024 · 3 comments · Fixed by #910
Assignees
Labels
enhancement New feature or request version-2.3

Comments

@mohamedselbohy
Copy link
Collaborator

Describe the bug:

threat numbers for next created threats are affected by just clicking unsaved threats.
Expected behavior:

It was expected for the new threats to stick to an incremental sequence when creating them and to be independent on unsaved threats the ones that get applied first get the next available threat number.
Environment:

  • Version: (2.2.0)
  • Platform: (Web App)

To Reproduce:

I believe the threat top number should only be affected when saving and creating a new threat. instead of affecting the numbers of new threats irrelevantly which may confuse end users.

Screenshot for clarification
image
threat 1 was created then saved then threat 0 was created and not applied then was clicked 3 times then threat 6 (should have been 2) was created and saved

@mohamedselbohy mohamedselbohy added the bug Something isn't working label Feb 28, 2024
@jgadsden jgadsden added enhancement New feature or request version-3.x and removed bug Something isn't working labels Feb 28, 2024
@jgadsden
Copy link
Collaborator

thanks @mohamedselbohy , not necessarily a bug because it is supposed to do this
but it is certainly an enhancement if you want to change the behavior

@jgadsden
Copy link
Collaborator

jgadsden commented Mar 4, 2024

Decided not go ahead with this, the desired behavior listed in #899 , so marking as "Won't Do"
Note: this decision has been reversed, see comment below

@jgadsden
Copy link
Collaborator

reinstating this issue, because it corrects a bug in the threat creation where the threat number is always initially '0'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request version-2.3
Projects
None yet
2 participants