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

Release: 13.8.0 #10108

Merged
merged 52 commits into from
Feb 26, 2023
Merged

Release: 13.8.0 #10108

merged 52 commits into from
Feb 26, 2023

Conversation

syuilo
Copy link
Member

@syuilo syuilo commented Feb 26, 2023

Summary

This is a release PR.

For more information on the release instructions, please see:
https://github.com/misskey-dev/misskey/blob/develop/CONTRIBUTING.md#release

For reviewers

  • CHANGELOGに抜け漏れは無いか
  • バージョンの上げ方は適切か
  • 他にこのリリースに含めなければならない変更は無いか
  • 全体的な変更内容を俯瞰し問題は無いか
  • レビューされていないコミットがある場合は、それが問題ないか
  • 最終的な動作確認を行い問題は無いか

などを確認し、リリースする準備が整っていると思われる場合は approve してください。

Checklist

  • package.jsonのバージョンが正しく更新されている
  • CHANGELOGが過不足無く更新されている
  • CIが全て通っている

atsu1125 and others added 30 commits February 24, 2023 13:08
* 起動時にRedisの疎通確認を行う

* check:connectをstart内に移動

---------

Co-authored-by: tamaina <[email protected]>
* fix(client): MkTime.timeにstringでもDateでない値が入った場合、?を表示

* fix(client): MkTimeを改良

* numberを許容

* falsyな値もとる

* 不明

* ありません

* fix
* fix(server): notes/createで、fileIdsと見つかったファイルの数が異なる場合はエラーにする

* NO_SUCH_FILE
* デプロイされているプレビュー環境がない場合はDestroy preview environmentを実行しないようにする

* CIがない場合の処理追加
* fix(client): Android ChromeでPWAとしてインストールできない問題を修正

* 順番関係ある?
* Update @swc/core to v1.3.36

* Update CHANGELOG.md

* Update CHANGELOG.md
* feat: 検索画面の UI を統一

* fix: エラーの修正

* add: changelog

---------

Co-authored-by: syuilo <[email protected]>
* Simplify search.vue

This is already handled by the code above it, no need to handle it twice

* Remove unused imports
@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

単純なRenoteができなくなってて草

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

Renoteできないのを直した

@syuilo syuilo merged commit 02c8fd9 into master Feb 26, 2023
@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

🙏🏻

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

RenoteでapiWithDialog出るのはうざいかも

(エラーだけ出す感じが良さそう)

@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

ふーむ

@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

そのうち慣れそう

@acid-chicken
Copy link
Member

モーダルじゃなくて Renote ボタンにエフェクトつけたら良さそうだけどまぁ今後でいいかなという気持ちだった

@mei23
Copy link
Contributor

mei23 commented Feb 26, 2023

CHANGELOG

notes/create のバリデーションとか入ってなくないかしら?

@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

ほんとだ

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

CHANGELOGにはリファクターを含めていなかったのでそういうのと同じで含めないものかと思っていた

@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

バリデーション機能していないのはバグじゃね

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

  • fix(server): notes/createで、fileIdsと見つかったファイルの数が異なる場合はエラーにする

↑これをバリデーションの問題だと勘違いしていたわ

@mei23
Copy link
Contributor

mei23 commented Feb 26, 2023

添付数とか字数制限突破するのはかなりのバグだと思うのだわ

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

そのうち慣れそう

しんどいかも

@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

そのうち慣れそう

しんどいかも

成功したかどうか分からないというのもつらいからしょうがない(エラーの時に出すだけだと、何も出なかった時に処理に時間がかかっているのか成功したのかが分からない)

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

成功したかどうか分からない

サウンドが鳴るのでそれで判断できていたんだけど

@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

iOSのブラウザだとサウンド鳴らないし、TLを表示してなかったら鳴らないこともありそう

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

TLを表示してなかったら鳴らないことも

これどこで鳴らしてるんだ?

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

iOSのブラウザだとサウンド鳴らない

しらねぇよ
Firefoxで使ってんだよ俺は

@syuilo
Copy link
Member Author

syuilo commented Feb 26, 2023

iOSのブラウザだとサウンド鳴らない

しらねぇよ Firefoxで使ってんだよ俺は

しらねぇよ
iOSで使ってんだよ俺は

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

…iOSのブラウザでサウンドが鳴らないこと自体がバグでは

@Khsmty
Copy link
Contributor

Khsmty commented Feb 26, 2023

サウンドが鳴るのでそれで判断できていたんだけど

モバイルで動画見ながら Misskey を見ると、音が鳴るごとに動画が止まるのでマスター音量を切ってます
なのでサウンド判断はつらいです
あと外出先とかで音を出せない状況もあると思います
(ついでですが、io で反応がないと RN できてるか不安という声もたまに見ます

@tamaina
Copy link
Contributor

tamaina commented Feb 26, 2023

とにかく不快なので、オプションで消すとかトーストにするとかにしたい

TLを表示してなかったら鳴らないことも

普通にTLから鳴ってた

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR ‼️ wrong locales This PR edits locales other than the ja-JP one. See locales/README.md
Projects
None yet
Development

Successfully merging this pull request may close these issues.