Skip to content
This repository has been archived by the owner on Mar 20, 2024. It is now read-only.

Issue/update #209

Merged
merged 4 commits into from
Mar 20, 2024
Merged

Issue/update #209

merged 4 commits into from
Mar 20, 2024

Conversation

ntsummers1
Copy link
Collaborator

@ntsummers1 ntsummers1 commented Mar 20, 2024

Summary

Fixes a bug with application submission.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

How To Test

Pull down and check application

Checklist:

  • My code follows the style guidelines of this project and has no linting errors
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation, including any applicable ADRs
  • My changes generate no new warnings
  • I have added tests that fail without these changes
  • New and existing tests (unit, integration, accessibility) pass locally
  • Documentation updated
  • [ ] If there are security concerns they are addressed or ticketed after being discussed

@ntsummers1 ntsummers1 self-assigned this Mar 20, 2024
@ntsummers1 ntsummers1 added the frontend For work that is primarily in the front end of the application label Mar 20, 2024
Copy link

codeclimate bot commented Mar 20, 2024

Code Climate has analyzed commit 0b373cf and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 94.4% (80% is the threshold).

This pull request will bring the total coverage in the repository to 95.7% (0.2% change).

View more on Code Climate.

@braxex braxex merged commit 41d5148 into main Mar 20, 2024
27 checks passed
@braxex braxex deleted the issue/update branch March 20, 2024 18:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
frontend For work that is primarily in the front end of the application
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants