Remove education dialog for error pills in Drafts list #6528
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why is this the best possible solution? Were any other approaches considered?
Error/no errors pills were introduced in Oct 2023. It's fairly common for data collectors to switch devices or otherwise need to reinstall Collect. Seeing the education message again is not a very big deal but it is slightly annoying so I think now is a good time to remove it.
Happy to discuss and close this if others see a good reason to keep it in longer.
I considered keeping the "New feature" string but that would require using it somewhere to pass lint and I don't think it's worth it. I can bring back the translations if we need it again.
I used the translation editor to remove the string definition from all languages. This left the comments behind and those will be cleaned up the next time we update from Transifex.
How does this change affect users? Describe intentional changes to behavior and behavior that could have accidentally been affected by code changes. In other words, what are the regression risks?
The intentional change is to remove the information dialog. I think there is a low risk of a bug. The biggest risk is probably that someone who hasn't yet seen the pills doesn't see the information dialog and is confused about their function. I think they're pretty self-explanatory so this feels unlikely to me.
Do we need any specific form for testing your changes? If so, please attach one.
No.
Does this change require updates to documentation? If so, please file an issue here and include the link below.
No.
Before submitting this PR, please make sure you have:
./gradlew connectedAndroidTest
(or./gradlew testLab
) and confirmed all checks still passDateFormatsTest