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

[#87] Update JS tests guide, and CSS classes guide. #88

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ioanacristinamarinescu
Copy link
Contributor

Scope

Fixes #87

Why we got into this (5 whys)

Mainly used for bugs. Can be removed for tasks or new features.

  • new conventions

Changes

Describe how the problem was fixed.

Briefly describe all changes made in a developer friendly way.

Briefly describe the end user changes in a tester friendly way.

List all changes for which an ideal solution was not found.
Describe the current solution.

What was not done yet and link to the ticket for the still to do work.

As a drive-by change add comments about out-of-scope minor drive-by changes.

How to try and test the changes

reviewers: @Robi-net

How the changes can be tested and verified by team members.
How to run manual tests.

  1. Title for first test case

For end-user changes, how the changes can be tested and verified by
our users.

List the steps to follow for checking that everything is OK.

Explicitly state the expected results.

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.

Update JS tests styleguide and CSS class naming conventions.
1 participant