Skip to content

fix: will panic when model contains a private field - #282

Merged
hwbrzzl merged 2 commits into
masterfrom
bowen/fix-struct-to-map
Aug 31, 2023
Merged

fix: will panic when model contains a private field#282
hwbrzzl merged 2 commits into
masterfrom
bowen/fix-struct-to-map

Conversation

@hwbrzzl

@hwbrzzl hwbrzzl commented Aug 31, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Closes #

📑 Description

✅ Checks

  • My pull request adheres to the code style of this project
  • My code requires changes to the documentation
  • I have updated the documentation as required
  • All the tests have passed

ℹ Additional Information

devhaozi
devhaozi reviewed Aug 31, 2023
View reviewed changes

@devhaozi devhaozi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix lint.

@codecov

codecov Bot commented Aug 31, 2023 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

Patch coverage is 81.81% of modified lines.

Files Changed Coverage
support/database/database.go 50.00%
database/gorm/event.go 75.00%
database/gorm/query.go 100.00%
database/gorm/utils.go 100.00%

📢 Thoughts on this report? Let us know!.

@hwbrzzl
hwbrzzl requested a review from devhaozi August 31, 2023 13:56
devhaozi
devhaozi approved these changes Aug 31, 2023
View reviewed changes

@devhaozi devhaozi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@hwbrzzl
hwbrzzl merged commit c38f2bf into master Aug 31, 2023
@hwbrzzl
hwbrzzl deleted the bowen/fix-struct-to-map branch August 31, 2023 14:19
hwbrzzl added a commit that referenced this pull request Dec 14, 2024
@hwbrzzl
feat: [#282] Print the values in ctx when calling facades.Log().WithC…
755a429 
…ontext(ctx).Info()
hwbrzzl added a commit that referenced this pull request Dec 15, 2024
@hwbrzzl
Merge branch 'master' into bowen/#282
c447238
hwbrzzl added a commit that referenced this pull request Dec 16, 2024
@hwbrzzl
Merge branch 'master' into bowen/#282
7be4485
hwbrzzl added a commit that referenced this pull request Dec 17, 2024
@hwbrzzl
Merge branch 'master' into bowen/#282
4a6524d
hwbrzzl added a commit that referenced this pull request Dec 17, 2024
@hwbrzzl
feat: [#282] Print the values in ctx when calling facades.Log().WithC…
9dddb62 
…ontext(ctx).Info() (#759)

* feat: [#282] Print the values in ctx when calling facades.Log().WithContext(ctx).Info()

* optimmize tests

* fix http.Request

* optimize first letter

* optimize lint
@krishankumar01 krishankumar01 mentioned this pull request Apr 29, 2026
Merged
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@krishankumar01 krishankumar01 Awaiting requested review from krishankumar01
1 more reviewer
@devhaozi devhaozi devhaozi approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

🚀 Review Ready

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@hwbrzzl @devhaozi