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

[MODFISTO-511] - Fix sql migration by using json_build_object #434

Merged
merged 4 commits into from
Nov 21, 2024

Conversation

azizbekxm
Copy link
Contributor

@azizbekxm azizbekxm commented Nov 19, 2024

Purpose

failied in run migration script
https://folio-org.atlassian.net/browse/MODFISTO-511

Approach

  • used jsonb || jsonb_build_object to bypass version lock system

@BKadirkhodjaev BKadirkhodjaev requested a review from a team November 21, 2024 09:42
@azizbekxm azizbekxm changed the title Fix sql migration by using json_build_object [MODFISTO-511] - Fix sql migration by using json_build_object Nov 21, 2024
@azizbekxm
Copy link
Contributor Author

testing results
generated data first
image

here is correct calculation need to be updated in budget
image

currently in budget
image

current encumbrance
image

after script run

image image

@azizbekxm azizbekxm requested a review from a team November 21, 2024 14:50
@azizbekxm azizbekxm merged commit c4dab7f into master Nov 21, 2024
8 checks passed
@azizbekxm azizbekxm deleted the MODFISTO-482 branch November 21, 2024 14:57
azizbekxm added a commit that referenced this pull request Nov 21, 2024
* Fix sql migration by using json_build_object

* Update sql query

* Update sql query

(cherry picked from commit c4dab7f)
azizbekxm added a commit that referenced this pull request Nov 21, 2024
* [MODFISTO-511] - Fix sql migration by using json_build_object (#434)

* Fix sql migration by using json_build_object

* Update sql query

* Update sql query

(cherry picked from commit c4dab7f)

* Update NEWS.md

* [MODINVOICE-563] Update encumbrance amount for unreleased status only (#435)

(cherry picked from commit 33f74c7)

* Update NEWS.md

* Update NEWS.md

* [maven-release-plugin] prepare release v8.7.2

* [maven-release-plugin] prepare for next development iteration

---------

Co-authored-by: Saba-Zedginidze-EPAM <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants