Thanks to visit codestin.com
Credit goes to github.com

Skip to content

将 Typecho 网站链接修改至 HTTPS 链接#1006

Closed
cherinyy wants to merge 3 commits intotypecho:masterfrom
cherinyy:master
Closed

将 Typecho 网站链接修改至 HTTPS 链接#1006
cherinyy wants to merge 3 commits intotypecho:masterfrom
cherinyy:master

Conversation

@cherinyy
Copy link

No description provided.

@Rainshaw
Copy link

Rainshaw commented Sep 1, 2020

为啥需要改啊,服务器上的SSL证书不是永久的,就应该用 http 的跳转呀。

@Mushan3420
Copy link

Mushan3420 commented Sep 1, 2020 via email

@Rainshaw
Copy link

Rainshaw commented Sep 1, 2020

@wordlesswind 已经都改过来了,@joyqi 点两下鼠标就能 merge,这咋和懒搭上边了。自动续期也有可能会失败:)

@Mushan3420
Copy link

Mushan3420 commented Sep 1, 2020 via email

@cherinyy
Copy link
Author

cherinyy commented Sep 1, 2020

@RainshawGao

自动续期也有可能会失败

我之前看了一下,网站用的是 Cloudflare CDN,所以自动续期是 Cloudflare 那边负责,一般不会出现续期失败的情况。

<a href="http://typecho.org" class="i-logo-s">Typecho</a>
<p><?php _e('由 <a href="http://typecho.org">%s</a> 强力驱动, 版本 %s (%s)', $options->software, $prefixVersion, $suffixVersion); ?></p>
<a href="https://typecho.org" class="i-logo-s">Typecho</a>
<p><?php _e('由 <a href="https://typecho.org">%s</a> 强力驱动, 版本 %s (%s)', $options->software, $prefixVersion, $suffixVersion); ?></p>
Copy link
Member

Choose a reason for hiding this comment

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

类似这种修改,会影响i18n的识别,导致翻译失败

Copy link
Author

Choose a reason for hiding this comment

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

类似这种修改,会影响i18n的识别,导致翻译失败

这样,那请问可以简单地将 typecho/languages 中的链接替换成相应的链接吗?

Copy link

Choose a reason for hiding this comment

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

似乎改动的工程有点大,你得把你发的那个文件夹里的都改了。。。

Copy link
Author

Choose a reason for hiding this comment

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

似乎改动的工程有点大,你得把你发的那个文件夹里的都改了。。。

如果可以简单地把 typecho/languages 中的链接给替换了,那倒简单,一键替换的事情。

@cherinyy cherinyy closed this Apr 15, 2021
@kecrily kecrily mentioned this pull request Jan 2, 2022
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.

4 participants