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

Update ux_issue.md #393

Merged
merged 1 commit into from
Mar 30, 2023
Merged

Update ux_issue.md #393

merged 1 commit into from
Mar 30, 2023

Conversation

lynwilhelm
Copy link
Contributor

Problem

What is the problem this work solves, including
Link to story or ticket

Solution

What I/we did to solve this problem

with @pairperson1

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • This change requires updated or new tests

Change summary:

  • Tidy, well formulated commit message
  • Another great commit message
  • Something else I/we did

Steps to Verify:

  1. A setup step / beginning state
  2. What to do next
  3. Any other instructions
  4. Expected behavior
  5. Suggestions for testing

Screenshots (optional):

Show-n-tell images/animations here

Keyfiles (delete if not relevant):

  1. main file/entry point
  2. other important file

Thanks for contributing!

@lynwilhelm lynwilhelm requested a review from a team as a code owner February 7, 2023 18:57
@lynwilhelm lynwilhelm requested review from toloudis, blairlyons and frasercl and removed request for a team February 7, 2023 18:57
@lynwilhelm
Copy link
Contributor Author

Slight change to template

@github-actions
Copy link

github-actions bot commented Feb 7, 2023

Coverage report

St.
Category Percentage Covered / Total
🟡 Statements 75.27% 557/740
🟡 Branches 70.64% 77/109
🔴 Functions 42.61% 75/176
🟡 Lines 73.86% 500/677

Test suite run success

90 tests passing in 7 suites.

Report generated by 🧪jest coverage report action from 16e9818

@meganrm meganrm merged commit 24637ff into main Mar 30, 2023
@meganrm meganrm deleted the lynwilhelm-patch-3 branch March 30, 2023 19:15
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.

3 participants