Skip to content

Sikoay #35

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

Merged
merged 2 commits into from
Jul 14, 2020
Merged

Sikoay #35

merged 2 commits into from
Jul 14, 2020

Conversation

Chen-Zhitong
Copy link
Contributor

在使用Admin时引入了OSS,之后发现出了一些问题,首先是mkdir()报错,以为是想要创建线上文件夹所以报错,结果后来发现此包中的临时存储位置在public中,改为框架推荐的storage中。

然后在新建文章时报错,发现初始image为空时会报错,于是做了小修复,如果之后再发现有问题会继续优化修改

@EVA09 EVA09 merged commit 9af669d into laravel-admin-extensions:master Jul 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants