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.

Sign up for GitHub

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

Jump to bottom

Remove concepts/mappings associations from collection versions #1206

Closed
snyaggarwal opened this issue Feb 9, 2022 · 5 comments
Closed

Remove concepts/mappings associations from collection versions #1206

snyaggarwal opened this issue Feb 9, 2022 · 5 comments
Assignees
@snyaggarwal
Labels
api2 OCL API v2

Comments

@snyaggarwal
Copy link
Contributor

Since expansions have concepts/mappings associations calculated from references on collection-version, we don't need old concepts/mappings associations on collection-versions.
@snyaggarwal snyaggarwal added the api2 OCL API v2 label Feb 9, 2022
@snyaggarwal snyaggarwal self-assigned this Feb 9, 2022
@snyaggarwal
Copy link
Contributor Author

@paynejd What will be the right time to pull the trigger on this? Before we move to staging?

@jamlung-ri
Copy link
Contributor

@snyaggarwal Ready when you are to put this on Staging!

@snyaggarwal
Copy link
Contributor Author

@jamlung-ri @paynejd Just realised that we cant remove this until all the environments collections are moved to newstyle.

snyaggarwal added a commit to OpenConceptLab/oclapi2 that referenced this issue Feb 25, 2022
@snyaggarwal
OpenConceptLab/ocl_issues#1206 | removed dead code
3daa48f
snyaggarwal added a commit to OpenConceptLab/oclapi2 that referenced this issue Apr 6, 2022
@snyaggarwal
OpenConceptLab/ocl_issues#1206 | removed code to migrate old style to…
1eeb11f 
… new style collection
snyaggarwal added a commit to OpenConceptLab/oclapi2 that referenced this issue Apr 6, 2022
@snyaggarwal
OpenConceptLab/ocl_issues#1206 | removed old collection concepts/mapp…
cbfa109 
…ings relations
@snyaggarwal
Copy link
Contributor Author

this is done and deployed on Staging. Will be going on to Production this week.

@snyaggarwal
Copy link
Contributor Author

This is deployed everywhere. Closing this out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@snyaggarwal snyaggarwal

Labels
api2 OCL API v2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@snyaggarwal @jamlung-ri