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

Fix typos in demo threat descriptions #951

Merged
merged 1 commit into from
Apr 18, 2024

Conversation

alexvorndran
Copy link
Contributor

Summary:

This pull request fixes some minor typos in the threat descriptions of the demo models (and a test fixture). These aren't critical in the sense that they'd distort the meaning, but the fix may help to improve the first impression of Threat Dragon.

Description for the changelog:

Fix typos in demo threat descriptions

Other info:

Based on my understanding of the end-to-end test that uses the fixture in question (td.vue/tests/e2e/specs/threatmodel/import.cy.js), the change shouldn't have any effect on the test because the changed description isn't checked. The test is also skipped at the moment.

@jgadsden jgadsden self-requested a review April 18, 2024 10:58
@jgadsden jgadsden added enhancement New feature or request version-2.3 labels Apr 18, 2024
Copy link
Collaborator

@jgadsden jgadsden left a comment

Choose a reason for hiding this comment

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

Thanks for this @alexvorndran , good to fix typos and going ahead with the merge

@jgadsden jgadsden merged commit b306e5a into OWASP:main Apr 18, 2024
@alexvorndran alexvorndran deleted the fix-typo-in-demo-models branch April 18, 2024 12:43
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
Development

Successfully merging this pull request may close these issues.

2 participants