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

ソフトウェアアップデート機能を実装 #264

Merged
merged 11 commits into from
Mar 20, 2021

Conversation

uzulla
Copy link
Collaborator

@uzulla uzulla commented Mar 16, 2021

ref: #39 #233

  • ソフトウェアアップデート機能を実装(管理画面「システム更新」より)
    • DB更新の発生しない範囲での更新が可能
    • index.phpを読み込むことで、app ディレクトリを自動判定し、更新できるように
    • GitHubのReleaseに記述することで、「システム更新」ページに、おしらせ的な文章を掲載可能(Releaseのタイトルと本文が出る)
    • (GitHub APIのRate limitがあるので、1hキャッシュを持つ)
  • 現状、 fc2blog/blog でリリースが打たれていないので、 uzulla/fc2blog を配布ソースとしている => fc2blog/blogに書き換えた
    • 現状、/fc2blog/blogリリースがないのでエラー表示になるが、それは正常
      • config.phpにてGITHUB_REPOを設定すればuzulla/fc2blogを参照し、(もし/app/temp/github_release_cache.jsonが存在するなら、削除してリロードすれば)テスト可能。
    • fc2blog/blog におけるタグ打ちの運用策定が必要 (インストーラー機能を実装 #259 (comment)
    • ( タグを私の方で代理で打ってよければ打ちます )
  • fc2blog_installer.phpの動作を改善し、再度設置することでインストール失敗リカバリを可能に修正
  • fc2blog_installer.phpのソースレポジトリをfc2blog/blogに変更した(今はリリースがないので、エラーになる。ファイルのコード先頭部分を/uzulla/fc2blogに書き換えでテスト可能

NOTE

  • ソフトウェア更新が必要な際の通知機能は未実装、別途実装

スクリーンショット

image

上はGITHUB_REPO/uzulla/fc2blogを指定してのテスト状態

(作業時間 8h

@uzulla
Copy link
Collaborator Author

uzulla commented Mar 16, 2021

現在Failしている以下箇所のテストは #263 で対処予定。(確率的に発生する)

FAIL tests/open_base_url.test.ts
  will be redirect to random blog page
    ✕ redirect to random blog page (438 ms)
    ✕ get Title after redirected page. (105 ms)

@uzulla uzulla mentioned this pull request Mar 16, 2021
5 tasks
- If you want continue to follow the uzulla's development version. Check `app/config.sample.php` and `fc2blog_installer.php`'s header.
@fc2dev fc2dev merged commit 056179d into fc2blog:master Mar 20, 2021
@fc2dev
Copy link
Contributor

fc2dev commented Mar 20, 2021

タグうちはお任せします。

@uzulla uzulla deleted the issue39/update-feature branch July 31, 2021 03:52
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