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

Update models.md to describe that non-exported fields are ignored #797

Open

maxant wants to merge 1 commit into go-gorm:master from maxant:patch-1

maxant commented Oct 23, 2024

include description that fields must be exported in order to be mapped

[] ONLY change English documents in the Pull Request, translations will be synced and translate them with https://translate.gorm.io/ or it will cause merge conflicts!

What did this pull request do?

schema.go doesn't inspect non-exported fields:

https://github.com/go-gorm/gorm/blob/deceebfab8c460cfee229233aded2821ac6b08eb/schema/schema.go#L204

see also go-gorm/gorm#7251


          Update models.md

46a590a

include description that fields must be exported in order to be mapped

maxant mentioned this pull request

rename column not working go-gorm/gorm#7251

Closed

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

Labels

None yet