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

fix: Assigning nil value to *uuid.UUID field in Updates #7099

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

fix: Assigning nil value to *uuid.UUID field in Updates #7099

wants to merge 3 commits into from

Commits on Sep 24, 2024

  1. fix: Assigning nil value to *uuid.UUID field in Updates 

    This PR adds handling for the assignment of a nil value to a *uuid.UUID field (resolved as a reflect.Ptr to a reflect.Array), to ensure that the model object reflects the correct value after Updates() has completed. This PR also adds few supporting test cases for Updates() with a map and uuid.UUID column.
    @omkar-foss
    omkar-foss committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    11ecd03 View commit details
    Browse the repository at this point in the history

  2. Add handling for embedded structs

    @omkar-foss
    omkar-foss committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    753049d View commit details
    Browse the repository at this point in the history

  3. Use gorm datatypes.UUID instead of uuid.UUID

    @omkar-foss
    omkar-foss committed Sep 24, 2024
    Configuration menu
    Copy the full SHA
    316303a View commit details
    Browse the repository at this point in the history