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 Crash When Clicking "+" to Add New Category #455

Merged
merged 1 commit into from
Dec 27, 2023

Conversation

mattttvaughn
Copy link
Contributor

On a fresh install, attempting to add a new category on the categories page would previously result in a crash, as the Edit Categories page didn't handle opening the screen without a category passed in.

This is mentioned in #441. This PR just resolves a minor portion of that ticket, the "add new category" that is mentioned as a footnote.

STRs:

  • Open app (fresh install)
  • Tap settings button
  • Tap Categories and Phrases
  • Tap + button

Expected:
The user is brought to a screen to add a new category.

Actual:
The app crashes.

Screenshots/videos:
Webm video of the app crashing when add new category clicked
Webm video of app opening the add new category page correctly

  • Acceptance Criteria satisfied
  • Regression Testing

Attempting to add a new category on the categories page would previously result in a crash, as the Edit Categories page didn't handle opening the screen without a category passed in.
Copy link
Collaborator

@PaulKlauser PaulKlauser left a comment

Choose a reason for hiding this comment

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

Ah, nice, this must have gotten missed when deleting duplicate Edit/New category Fragments

@mattttvaughn mattttvaughn merged commit 77f4ff4 into main Dec 27, 2023
4 checks passed
@mattttvaughn mattttvaughn deleted the fix-crash-on-add-category branch December 27, 2023 14:31
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.

2 participants