-
Notifications
You must be signed in to change notification settings - Fork 2
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
Performance | Repo HEAD to not contain all resource versions. #1262
Comments
… versions but latest only
…ly with resource latest versions
…rsions associations
@jamlung-ri @paynejd One Source HEAD on QA is migrated -- Please take a look (there shouldn't be any difference) |
@snyaggarwal I have done a spot check of about 10 concepts, and then I did a version export and validated that using the script. Everything seems good. Next step is to migrate a more complicated source with concept versions: SNOMED in QA. Here is an example concept that has multiple versions: https://app.qa.openconceptlab.org/#/orgs/International-Health-Terminology-Standard-Development-Organisation-IHTSDO-/sources/SNOMED/concepts/767524001/ After that, we should create a more complicated source with lots of concept versions and source versions, and then we can update that. |
@snyaggarwal I compared HEAD with the previous SNOMED version and could not find any differences between the two. I can't tell that anything has changed between the versions. Hopefully that's a good thing? |
@jamlung-ri Yes thats good |
ready to apply to all of QA?
…On Wed, Apr 6, 2022 at 2:46 AM Sny ***@***.***> wrote:
@jamlung-ri <https://github.com/jamlung-ri> Yes thats good
—
Reply to this email directly, view it on GitHub
<#1262 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJCOOP2VOFS327BPNOIEALVDUXL5ANCNFSM5RZ5XVTA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@jamlung-ri @paynejd Applied on all Sources in QA. |
@paynejd Is it ok to apply this on entire Staging? |
Let’s do it!
…On Fri, Apr 8, 2022 at 2:55 AM Sny ***@***.***> wrote:
@paynejd <https://github.com/paynejd> Is it ok to apply this on entire
Staging?
—
Reply to this email directly, view it on GitHub
<#1262 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAJCOOPBNCKSZEGDUAQLJO3VD7J7HANCNFSM5RZ5XVTA>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
@paynejd Triggered on Staging |
Need to do testing on this, probably verifying some sources on Staging next week |
…in list queries
@paynejd This is deployed everywhere. Closing this out |
No description provided.
The text was updated successfully, but these errors were encountered: