Skip to content

fix(formulus): Fix app bundle button enabled/disabled confusion#555

Merged
r0ssing merged 2 commits intoOpenDataEnsemble:devfrom
HelloSapiens:fix/update-app-bundle-button
Mar 21, 2026
Merged

fix(formulus): Fix app bundle button enabled/disabled confusion#555
r0ssing merged 2 commits intoOpenDataEnsemble:devfrom
HelloSapiens:fix/update-app-bundle-button

Conversation

@r0ssing
Copy link
Contributor

@r0ssing r0ssing commented Mar 21, 2026

Sync screen: Update App Bundle button

Description

This PR fixes a race condition on the sync screen when identifying whether there is an updated app bundle available.

Type of Change

  • Bug Fix
  • New Feature / Enhancement
  • Refactor / Code Cleanup
  • Documentation Update
  • Maintenance / Chore
  • Other (please specify):

Component(s) Affected

  • formulus (React Native mobile app)
  • formulus-formplayer (React web app)
  • synkronus (Go backend server)
  • synkronus-cli (Command-line utility)
  • Documentation
  • DevOps / CI/CD
  • Other:

Testing

  • Unit tests added/updated
  • Integration tests added/updated
  • Manually tested
  • Tested on multiple platforms (if applicable)
  • Not applicable

Breaking Changes

  • This PR introduces breaking changes
  • This PR does NOT introduce breaking changes

If breaking changes, please describe migration steps:


Documentation Updates

  • Documentation has been updated
  • Documentation update is not required

Checklist

  • Code follows project style guidelines
  • All existing tests pass
  • New tests added for new functionality
  • PR title follows Conventional Commits format

Thank you for contributing to Open Data Ensemble (ODE)!

r0ssing added 2 commits March 21, 2026 10:23
…yncService

- Introduced `normalizeAppBundleVersion` utility to standardize app version formatting.
- Updated version checks in SyncScreen and SyncService to use normalized versions for consistency.
- Enhanced sync logic to refresh bundle status appropriately based on sync state changes.
Copy link
Contributor

@drcelsobat drcelsobat left a comment

Choose a reason for hiding this comment

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

lgtm

@r0ssing r0ssing merged commit eb7fdac into OpenDataEnsemble:dev Mar 21, 2026
10 checks passed
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