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

HPC-9997: Simplify support array in PLAN_ENTITY_VERSION_VALUE codec #202

Merged
merged 2 commits into from
Jan 16, 2025

Conversation

Pl217
Copy link
Contributor

@Pl217 Pl217 commented Dec 30, 2024

Note

To be merged after admin command from https://github.com/UN-OCHA/hpc_service/pull/3684 is run

After fixing database records so that support array is initialized everywhere and it can no longer have object with key "0", we can simplify the codec PLAN_ENTITY_VERSION_VALUE

@Pl217 Pl217 added ready for review All comments have been addressed, and the Pull Request is ready for review pending prior merge Another Pull Request needs to be merged before this one labels Dec 30, 2024
@Pl217 Pl217 requested a review from a team as a code owner December 30, 2024 19:48
@enxtur enxtur assigned Pl217 and unassigned enxtur Jan 13, 2025
@enxtur enxtur added ready for merge Review and testing is complete. It is ready for merging as soon as CI has finished. and removed ready for review All comments have been addressed, and the Pull Request is ready for review labels Jan 13, 2025
Pl217 and others added 2 commits January 16, 2025 23:24
After fixing database records so that `support`
array is initialized everywhere and it can no
longer have object with key `"0"`, we can simplify
the codec `PLAN_ENTITY_VERSION_VALUE`
@enxtur enxtur merged commit fc9aa70 into develop Jan 16, 2025
2 checks passed
@enxtur enxtur deleted the HPC-9997 branch January 16, 2025 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending prior merge Another Pull Request needs to be merged before this one ready for merge Review and testing is complete. It is ready for merging as soon as CI has finished.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants