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

Migration to populate filledBy #694

Open
mahalakshme opened this issue Feb 22, 2024 · 2 comments
Open

Migration to populate filledBy #694

mahalakshme opened this issue Feb 22, 2024 · 2 comments
Labels

Comments

@mahalakshme
Copy link
Contributor

mahalakshme commented Feb 22, 2024

Need:

  • Needed to be able to write report card queries so that it doesn't miss out displaying any subjects in the result

Acceptance criteria:

  • For the org in focus alone(Teach AP app), add migration to populate filledBy field with lastModifiedBy field in server for encounter table.

Out of scope:

  • No need to do anything for createdBy and lastModifiedBy related audit fields since they will be handled by fresh sync.
    createdBy, lastModifiedBy and filledBy fields in the client will get populated when users do fresh sync when the new infra is setup

Tech notes:

  • Not flyway migration
@mahalakshme mahalakshme transferred this issue from avniproject/avni-client Feb 22, 2024
@mahalakshme
Copy link
Contributor Author

Moving to Hold since need confirmation from the client if the above approach is fine. Same will be discussed during showcase.

@petmongrels
Copy link
Contributor

accidentally picked up. WIP query has been checked in here -
https://github.com/avniproject/data-fixes/blob/d0e0a379a484cb6a5c696c65541fe7145db2f946/lfe/filledBy.sql

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Analysis Complete
Development

No branches or pull requests

3 participants