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

Reset Sync flow broken after switch to Embedded dataType #1295

Closed
2 tasks
himeshr opened this issue Feb 9, 2024 · 3 comments
Closed
2 tasks

Reset Sync flow broken after switch to Embedded dataType #1295

himeshr opened this issue Feb 9, 2024 · 3 comments
Assignees
Labels

Comments

@himeshr
Copy link
Contributor

himeshr commented Feb 9, 2024

Describe the bug
I do not have clear instructions to reproduce different issues encountered during reset-sync.
But as of now, i have captured errorLogs / stack-trace of 2 issues.

Issue 1: When resetSync is triggered, app throws error "Exception in HostFunction: You cannot query an embedded object."
Issue 2: When we change Catchment for user AchalaB@power, from "test" to "test new", then on sync, app throws error "lowestAddressLevel are mandatory for Individual, Keys being saved - ...."

User: AchalaB@power
Env: Prerelease
Apk: local run_app_pre-release dev on 7.0 branch

Expected behavior
Reset sync should work in various scenarios such as

  1. Catchment change
  2. SyncConcept value change for user
  3. SyncConcept removal / addition for SubjectType

Screenshots
Screenshot 2024-02-09 at 6 31 06 PM

Logs
ResetSyncLogs2ndIssue.txt
ResetSyncLogs.txt

Developer Checklist
Developer fixing the bug should fill this checklist.

  • Does the fix require extensive regression testing?
  • Are you mentioning the required scenarios that could be affected?
@himeshr himeshr self-assigned this Feb 9, 2024
@github-project-automation github-project-automation bot moved this to New Issues in Avni Product Feb 9, 2024
@himeshr himeshr moved this from New Issues to Ready in Avni Product Feb 9, 2024
@himeshr himeshr added the bug label Feb 9, 2024
@himeshr himeshr moved this from Ready to In Progress in Avni Product Feb 9, 2024
himeshr added a commit that referenced this issue Feb 12, 2024
@himeshr
Copy link
Contributor Author

himeshr commented Feb 12, 2024

Validated the test scenarios as part of dev-testing:

  1. Catchment change
  2. SyncConcept value change for user
  3. SyncConcept removal / addition for SubjectType

Couldn't reproduce the 2nd issue mentioned here, fixed 1st one.

@himeshr himeshr moved this from In Progress to Code Review Ready in Avni Product Feb 12, 2024
himeshr added a commit to avniproject/avni-models that referenced this issue Feb 12, 2024
…gure out embedded entities to avoid coding miss when creating new Embedded objects
@petmongrels petmongrels moved this from Code Review Ready to In Code Review in Avni Product Feb 13, 2024
@petmongrels petmongrels moved this from In Code Review to QA Ready in Avni Product Feb 13, 2024
@Dinesh2019
Copy link

Dinesh2019 commented Feb 14, 2024

Still, we are facing this issue

Env: pre-release
org: rwb24uat
user: dinesh@rwb24uat
apk : https://app.circleci.com/pipelines/github/avniproject/avni-client/2316/workflows/4790a570-ab62-4209-9709-b8e01d61ccea/jobs/14418/artifacts

steps to reproduce:

  1. Login to the org
  2. Wait still sync got completed
  3. go to the web app
  4. Under user update the catchment of the user
  5. again come back to the app
  6. click on the sync

Expected: sync should be completed
Actual: sync displays the error

Reference video:

Screen.Recording.2024-02-14.at.1.22.26.PM.mov

@Dinesh2019 Dinesh2019 moved this from QA Ready to QA Failed in Avni Product Feb 14, 2024
@himeshr
Copy link
Contributor Author

himeshr commented Feb 14, 2024

@Dinesh2019 the apk used is from master branch, but it doesnot have the latest code changes done in 7.0 branch to fix Embedded entity related issues.

Use this apk for 7.0 release cards testing.
https://app.circleci.com/pipelines/github/avniproject/avni-client/2314/workflows/bf70df9e-92ae-4826-9215-9252a91a0214/jobs/14419/artifacts

@himeshr himeshr moved this from QA Failed to QA Ready in Avni Product Feb 14, 2024
@Dinesh2019 Dinesh2019 moved this from QA Ready to In QA in Avni Product Feb 14, 2024
@mahalakshme mahalakshme moved this from In QA to Done in Avni Product Feb 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

3 participants