Skip to content

updated the Build of UI#987

Merged
bharadwaj-aditya merged 9 commits intoGoogleCloudPlatform:masterfrom
ollionorg:ui_build
Jan 17, 2025
Merged

updated the Build of UI#987
bharadwaj-aditya merged 9 commits intoGoogleCloudPlatform:masterfrom
ollionorg:ui_build

Conversation

@VivekY1098
Copy link
Contributor

Fixes #<issue_number_goes_here>

  1. Updated the UI build

It's a good idea to open an issue first for discussion.

  • Tests pass
  • Appropriate changes to README are included in PR

taherkl and others added 8 commits January 2, 2025 10:44
…INTS (GoogleCloudPlatform#978) (#30)

* verification ap and dump flow changes

* fixed IT issue

* Check constraints verificartion api v2 (#24)

* handled function not found

* added unhandled error

* updated the error msg

---------



* fix IT issue

* comment addressed (#27)

* comment addressed
1. rename the functionNotFound
2. added condition to call verification api

* spell checked

---------



* refactor the DbDumpImpl struct (#28)

* refactor the DbDumpImpl struct

* remove the GenerateCheckConstrainstExprId method

---------



* fixed if condition

---------

Co-authored-by: Taher Lakdawala <78196491+taherkl@users.noreply.github.com>
Co-authored-by: taherkl <taher.lakdawala@ollion.com>
Co-authored-by: Vivek Yadav <vivek.yadav@ollion.com>
Co-authored-by: Vivek Yadav <105432992+VivekY1098@users.noreply.github.com>
* Replace ReadOnlyTransaction() with Single() (GoogleCloudPlatform#984)

* UI changes for check constraint support (GoogleCloudPlatform#979)

* UI changes for check constraint support

* removed url changed for testing

* Check UI cmt (#29)

* remove the block which will not execute

* refactor the code
1. remove the unreachable code
2.  remvoe the node-sql-parser package

* refactor the code
1. rename the generateId method

---------

Co-authored-by: Vivek Yadav <vivek.yadav@ollion.com>

* updated the dist file and package.lock file

* Develop (#36)

* verification API and dump flow changes to support MySQL CHECK CONSTRAINTS (GoogleCloudPlatform#978) (#30)

* verification ap and dump flow changes

* fixed IT issue

* Check constraints verificartion api v2 (#24)

* handled function not found

* added unhandled error

* updated the error msg

---------



* fix IT issue

* comment addressed (#27)

* comment addressed
1. rename the functionNotFound
2. added condition to call verification api

* spell checked

---------



* refactor the DbDumpImpl struct (#28)

* refactor the DbDumpImpl struct

* remove the GenerateCheckConstrainstExprId method

---------



* fixed if condition

---------

Co-authored-by: Taher Lakdawala <78196491+taherkl@users.noreply.github.com>
Co-authored-by: taherkl <taher.lakdawala@ollion.com>
Co-authored-by: Vivek Yadav <vivek.yadav@ollion.com>
Co-authored-by: Vivek Yadav <105432992+VivekY1098@users.noreply.github.com>

* Replace ReadOnlyTransaction() with Single() (GoogleCloudPlatform#984) (#35)

Co-authored-by: Manit Gupta <manitgupta@google.com>

---------

Co-authored-by: Taher Lakdawala <78196491+taherkl@users.noreply.github.com>
Co-authored-by: Akash Thawait <aakash@ollion.com>
Co-authored-by: taherkl <taher.lakdawala@ollion.com>
Co-authored-by: Vivek Yadav <vivek.yadav@ollion.com>
Co-authored-by: Manit Gupta <manitgupta@google.com>

---------

Co-authored-by: taherkl <taher.lakdawala@ollion.com>
Co-authored-by: Akash Thawait <aakash@ollion.com>
Co-authored-by: Vivek Yadav <105432992+VivekY1098@users.noreply.github.com>
Co-authored-by: Vivek Yadav <vivek.yadav@ollion.com>
Co-authored-by: Manit Gupta <manitgupta@google.com>

---------

Co-authored-by: Manit Gupta <manitgupta@google.com>
Co-authored-by: Taher Lakdawala <78196491+taherkl@users.noreply.github.com>
Co-authored-by: taherkl <taher.lakdawala@ollion.com>
Co-authored-by: Vivek Yadav <105432992+VivekY1098@users.noreply.github.com>
Co-authored-by: Vivek Yadav <vivek.yadav@ollion.com>
@VivekY1098 VivekY1098 requested a review from a team as a code owner January 16, 2025 07:08
@VivekY1098 VivekY1098 requested review from darshan-sj and shreyakhajanchi and removed request for a team January 16, 2025 07:08
@VivekY1098 VivekY1098 closed this Jan 16, 2025
@VivekY1098 VivekY1098 reopened this Jan 16, 2025
@VivekY1098 VivekY1098 changed the title UI build updated the Build of UI Jan 16, 2025
@bharadwaj-aditya bharadwaj-aditya merged commit 85d34ba into GoogleCloudPlatform:master Jan 17, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants