fix: only show changelog after update, not on first install #3363
+1
−1
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.
Summary
Fixes the changelog tab appearing when the app is first opened after download. The changelog should only show after an actual update, not on first install.
The bug was in
maybe_emit_updated()- when there's no storedlast_seen_version(first install), it was emitting the updated event withprevious: None, which caused the changelog tab to open. Changed this to not emit the event on first install, while still saving the current version for future update detection.Review & Testing Checklist for Human
Notes
Requested by @ComputelessComputer
Link to Devin run: https://app.devin.ai/sessions/ea84fde78ae047e2a21c0b7df4beacd3