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

整合 API #144

Merged
merged 6 commits into from Dec 3, 2019
Merged

整合 API #144

merged 6 commits into from Dec 3, 2019

Conversation

spirit1431007
Copy link
Collaborator

No description provided.

@spirit1431007
Copy link
Collaborator Author

我在考虑要不要把图片移到images文件夹去,像这样:

├── images
│   ├── feature
│   ├── cover   

@mashirozx
Copy link
Owner

我觉得其实做成类似背景图列表那样自己填url会更好

@mashirozx
Copy link
Owner

早年一直想写一个python脚本,把图转成webp,生成一个manifest.json,都上传到jsdelivr,博客这边直接通过manifest.json获取资源路径,同时还能做一下webp自适应 https://2heng.xin/2018/03/21/webp-support-with-php-and-python/

不过懒一直没做。。

@spirit1431007
Copy link
Collaborator Author

这个好,我去研究研究

cover/img.txt Outdated
@@ -0,0 +1,3 @@
https://cdn.jsdelivr.net/gh/mashirozx/Sakura@3.3.2/cover/gallery/pixar_up_movie_balloons_desktop_1920x1080_hd-wallpaper-507967.jpg
https://cdn.jsdelivr.net/gh/mashirozx/Sakura@3.3.2/cover/gallery/beyond_by_auroralion-dan5njb.jpg
https://cdn.jsdelivr.net/gh/mashirozx/Sakura@3.3.2/cover/gallery/海の空(56993195)_by_夏T.jpg
Copy link
Owner

Choose a reason for hiding this comment

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

我的想法是这一块最好不要依赖读写本地文件实现,因为用户不一定方便修改这些东西,所以要么写在数据库里,要么写在外部储存里

'name' => __('Jsdelivr CDN', 'sakura'),/*图片库*/
'desc' => __('Fill in the cdn path for random picture display, without adding a slash at the end, for example: https://cdn.jsdelivr.net/gh/mashirozx/sakura@3.3.3, please refer to <a href = "https: //github.com/mashirozx/Sakura/wiki/options">Wiki </a>', 'sakura'),/*填写 cdn 路径,用于随机图片展示,最后不用加斜杠,例:https://cdn.jsdelivr.net/gh/mashirozx/sakura@3.3.3 ,更多信息请参考<a href="https://github.com/mashirozx/Sakura/wiki/options">Wiki</a>*/
'id' => 'jsdelivr_cdn',
'std' => 'https://cdn.jsdelivr.net/gh/mashirozx/sakura@3.3.3',
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

注意:下次release版本号需为v3.3.3

@mashirozx
Copy link
Owner

可以顺便把manifest里面的图片换成以前cover里面的图吗,昨天我忘记换了

@spirit1431007
Copy link
Collaborator Author

嗯,我直接在dev分支push吧,以免出错

@mashirozx
Copy link
Owner

好的

@mashirozx
Copy link
Owner

改了一下manifest上传接口路由
sakura/v1/image/manifest

@mashirozx mashirozx merged commit f301cdc into mashirozx:dev Dec 3, 2019
mashirozx pushed a commit that referenced this pull request Aug 7, 2021
Bumps [@formatjs/intl](https://github.com/formatjs/formatjs) from 1.13.2 to 1.13.4.
- [Release notes](https://github.com/formatjs/formatjs/releases)
- [Commits](https://github.com/formatjs/formatjs/compare/@formatjs/intl@1.13.2...@formatjs/intl@1.13.4)

---
updated-dependencies:
- dependency-name: "@formatjs/intl"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

None yet

2 participants