Skip to content

fix(desktop): honor includeDeleted in observation queries - #944

Merged
r0ssing merged 2 commits into
OpenDataEnsemble:devfrom
sarahmirembe592:fix/910-include-deleted
Oct 1, 2026
Merged

r0ssing merged 2 commits into
OpenDataEnsemble:devfrom
sarahmirembe592:fix/910-include-deleted

Conversation

@sarahmirembe592

@sarahmirembe592 sarahmirembe592 commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

What & Why

Fixed the desktop observation query to respect includeDeleted

Type of Change

  • Bug fix
  • New feature / enhancement
  • Refactor / cleanup
  • Documentation
  • Chore / other

Component(s) Changed

  • formulus (React Native mobile app)
  • formulus-formplayer (form WebView UI)
  • synkronus (Go backend)
  • synkronus-cli (command-line client)
  • synkronus-portal (web admin UI)
  • desktop (Tauri app)
  • packages (tokens / components)
  • Documentation / DevOps / CI

Testing

  • Added/updated tests
  • Manually tested
  • Not applicable

Checklist

  • Pre-flight checks for changed packages passed (lint / format / tests)
  • Breaking changes (if any) have migration steps
  • Related issue linked, Closes #910

@sarahmirembe592
sarahmirembe592 marked this pull request as draft September 28, 2026 22:15
@sarahmirembe592
sarahmirembe592 marked this pull request as ready for review September 28, 2026 22:17

@r0ssing r0ssing left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks great! Thank you @sarahmirembe592 - I will merge this now, so it will be included in v1.4.0. Welcome to ODE! 🦒

@r0ssing
r0ssing merged commit eacb98e into OpenDataEnsemble:dev Oct 1, 2026
12 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