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

1.文件存储增加minio驱动支持 2.优化文件存储minio驱动Url()方法 #48

Merged
merged 7 commits into from
Feb 22, 2023

Conversation

hongyukeji
Copy link
Contributor

#47 没改成功Url()方法, 这回这个ok了

Copy link
Contributor

@hwbrzzl hwbrzzl left a comment

Choose a reason for hiding this comment

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

Perfect, just a few tips, thanks for your PR.

filesystem/minio.go Show resolved Hide resolved
filesystem/minio.go Outdated Show resolved Hide resolved
filesystem/minio.go Show resolved Hide resolved
filesystem/minio.go Show resolved Hide resolved
filesystem/minio.go Outdated Show resolved Hide resolved
filesystem/minio.go Show resolved Hide resolved
filesystem/minio.go Outdated Show resolved Hide resolved
filesystem/minio.go Show resolved Hide resolved
filesystem/minio.go Outdated Show resolved Hide resolved
filesystem/minio.go Outdated Show resolved Hide resolved
@hwbrzzl hwbrzzl merged commit f4188ec into goravel:master Feb 22, 2023
dragoonchang added a commit to dragoonchang/framework that referenced this pull request Apr 11, 2023
dragoonchang added a commit to dragoonchang/framework that referenced this pull request Apr 11, 2023
hwbrzzl added a commit that referenced this pull request Apr 13, 2023
* Add Gorm.Expr()

* Implement db.Raw() as alias of gorm.Expr()

* Update database/db/db.go

Co-authored-by: Wenbo Han <wenbo.han@compass.com>

* Implement db.Raw() as alias of gorm.Expr()

* Update gorm_test.go

* Add facades.Config.Set method #48

* Add facades.Config.Set method #48

* Add test case for handling nested maps in config

---------

Co-authored-by: Alex Chang <alex.chang@techdesign.com>
Co-authored-by: Wenbo Han <wenbo.han@compass.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants