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.9.2 #10218

Merged
merged 53 commits into from
Mar 6, 2023
Merged

Release: 13.9.2 #10218

merged 53 commits into from
Mar 6, 2023

Conversation

syuilo
Copy link
Member

@syuilo syuilo commented Mar 5, 2023

No description provided.

syuilo and others added 30 commits March 4, 2023 10:17
…in) (#10190)

* Revert "Revert "fix(server): DriveFile related N+1 query when call note packMany (#10133)""

This reverts commit a7c82ee.

* packManyByIdsMap: 存在チェックをしてなかったものは null を入れるように

* Note.packMany で reply とか renote がもうあったらそのファイルも引く

* テストを書く

* fix test

* fix test

* fix test

* fix test
* enhance(server): downloadUrlでContent-Dispositionからファイル名を取得
Resolve #10036
Resolve #4750

* untitled

* オブジェクトストレージのContent-Dispositionのファイル名の拡張子をContent-Typeに添ったものにする

* ✌️

* tiff

* fix filename

* add test

* /files/でもContent-Disposition

* comment

* fix test
…ge indicate this (#10191)

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
* enhance(server): downloadUrlでContent-Dispositionからファイル名を取得
Resolve #10036
Resolve #4750

* untitled

* オブジェクトストレージのContent-Dispositionのファイル名の拡張子をContent-Typeに添ったものにする

* ✌️

* tiff

* fix filename

* add test

* /files/でもContent-Disposition

* enhance(server): メディアプロキシでico,bmpを読めるように
Fix #10120

* comment

* fix test

---------

Co-authored-by: syuilo <Syuilotan@yahoo.co.jp>
* docker: build assets on native arch

* 消しすぎた
* New translations ja-JP.yml (Chinese Simplified)

* New translations ja-JP.yml (Japanese, Kansai)

* New translations ja-JP.yml (Italian)

* New translations ja-JP.yml (Japanese, Kansai)

* New translations ja-JP.yml (Lao)

* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (Lao)

* New translations ja-JP.yml (German)

* New translations ja-JP.yml (Italian)

* New translations ja-JP.yml (English)

* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (German)

* New translations ja-JP.yml (German)

* New translations ja-JP.yml (English)

* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (Lao)

* New translations ja-JP.yml (Lao)

* New translations ja-JP.yml (Japanese, Kansai)

* New translations ja-JP.yml (Chinese Traditional)
* New translations ja-JP.yml (German)

* New translations ja-JP.yml (English)

* New translations ja-JP.yml (Japanese, Kansai)
@github-actions github-actions bot added packages/backend Server side specific issue/PR packages/frontend Client side specific issue/PR 🧪Test labels Mar 5, 2023
@codecov
Copy link

codecov bot commented Mar 5, 2023

Codecov Report

Merging #10218 (5ccbeda) into master (7e87005) will increase coverage by 0.09%.
The diff coverage is 64.28%.

❗ Current head 5ccbeda differs from pull request most recent head 31f9ea3. Consider uploading reports for the commit 31f9ea3 to get more accurate results

@@            Coverage Diff             @@
##           master   #10218      +/-   ##
==========================================
+ Coverage   69.14%   69.23%   +0.09%     
==========================================
  Files         706      707       +1     
  Lines       65312    65407      +95     
  Branches     5181     5214      +33     
==========================================
+ Hits        45159    45286     +127     
+ Misses      20153    20121      -32     
Impacted Files Coverage Δ
...kend/src/core/entities/GalleryPostEntityService.ts 51.66% <0.00%> (-0.88%) ⬇️
packages/backend/src/server/FileServerService.ts 19.05% <5.71%> (-0.51%) ⬇️
packages/backend/src/core/DriveService.ts 57.08% <52.00%> (-0.06%) ⬇️
...ges/backend/src/core/entities/UserEntityService.ts 91.60% <57.14%> (ø)
packages/backend/src/core/DownloadService.ts 71.62% <73.68%> (+0.08%) ⬆️
...ackend/src/core/entities/DriveFileEntityService.ts 71.18% <100.00%> (+2.24%) ⬆️
...ges/backend/src/core/entities/NoteEntityService.ts 83.25% <100.00%> (+0.88%) ⬆️
packages/backend/src/misc/correct-filename.ts 100.00% <100.00%> (ø)
packages/backend/src/misc/is-mime-image.ts 100.00% <100.00%> (ø)
... and 3 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rinsuki
Copy link
Contributor

rinsuki commented Mar 5, 2023

ふと思ったけど identicon のキー変えると生成される画像変わるのでちょっと微妙かも

@rinsuki
Copy link
Contributor

rinsuki commented Mar 5, 2023

(せめて CHANGELOG にはその旨但し書きしておいたほうがよさそう)

@Khsmty
Copy link
Contributor

Khsmty commented Mar 5, 2023

新しく追加されたチャンネルの投稿ボタンから投稿すると、リクエストに channelId が乗らずに普通の投稿になってしまいます
image

@syuilo
Copy link
Member Author

syuilo commented Mar 5, 2023

新しく追加されたチャンネルの投稿ボタンから投稿すると、リクエストに channelId が乗らずに普通の投稿になってしまいます image

👍🏻👍🏻👍🏻

直しました

@tamaina
Copy link
Contributor

tamaina commented Mar 5, 2023

しゅいろが寝る直前にリリースを切る習慣をやめて欲しいので個人的には明日の朝Approveしたい所存

Comment on lines +662 to +667
text.includes('https://youtu.be/Efrlqw8ytg4'.toLowerCase()) ||
text.includes('https://www.youtube.com/watch?v=Efrlqw8ytg4'.toLowerCase()) ||
text.includes('https://m.youtube.com/watch?v=Efrlqw8ytg4'.toLowerCase()) ||
text.includes('https://youtu.be/XVCwzwxdHuA'.toLowerCase()) ||
text.includes('https://www.youtube.com/watch?v=XVCwzwxdHuA'.toLowerCase()) ||
text.includes('https://m.youtube.com/watch?v=XVCwzwxdHuA'.toLowerCase())
Copy link
Member

Choose a reason for hiding this comment

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

toLowerCase は嘘そう

Copy link
Contributor

Choose a reason for hiding this comment

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

なくても良さそうではある

@@ -6766,14 +6789,16 @@ packages:
readable-stream: 2.3.7
dev: false

/follow-redirects/1.15.2:
/follow-redirects/1.15.2_debug@4.3.4:
Copy link
Member

Choose a reason for hiding this comment

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

なんだろうこれ

Copy link
Contributor

Choose a reason for hiding this comment

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

私の環境でpnpm addとかすると勝手につく

tamaina and others added 5 commits March 6, 2023 01:17
* New translations ja-JP.yml (Thai)

* New translations ja-JP.yml (Chinese Simplified)

* New translations ja-JP.yml (Italian)

* New translations ja-JP.yml (Italian)
@tamaina
Copy link
Contributor

tamaina commented Mar 6, 2023

Approveしたけどビルドが…

ci skipしたほうが良かったかもしれない

@tamaina
Copy link
Contributor

tamaina commented Mar 6, 2023

Dockerビルド、激早になってた

@syuilo syuilo merged commit ae517a9 into master Mar 6, 2023
@syuilo
Copy link
Member Author

syuilo commented Mar 6, 2023

🙏🏻

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.