Repository navigation
Conversation
Android の通知が「alice / リアクション :igyo:」のように種別だけで、何のノートに
ついたのか通知を見ただけでは分からなかった。本家 Web (packages/sw の
create-notification.ts) と同じく、見出しに「誰が何をしたか」、本文に対象ノートの
本文を出す。
- reaction: 見出し「👍 名前」(カスタム絵文字はコロンとホストを外す)、本文 = ノート本文
- mention / reply / quote / note: 見出し「{name}からのメンション」等、本文 = ノート本文
- renote: 本文はリノートされた元ノート (通知の note はリノート側)
- follow 系: 見出しが固定文、本文が相手の名前
- CW 付きは本文ではなく注釈を出す (ロック画面に伏せた本文を出さない)。
isHidden と空文は本文なし
- Android は BigTextStyle で展開時に全文を折り返し、絵文字画像があるときは
BigPictureStyle の summary に本文を載せる
- デスクトップのまとめ通知の名前列挙は見出しではなく NotifyPlan.actor_name から
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
Android は要約通知を作らず OS 任せなので、まとめ用の名前は使わない。 CI の Clippy (app, Android) が -D warnings で落ちていた。 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
本家 backend は cw を minLength 1 で trim せずに保存し、Web は cw != null で 本文を隠す。CW が空白だけのとき本文へフォールバックしていたので、Web では 隠れている本文がロック画面に出ていた。CW が Some なら本文には落とさない。 まとめ通知の名前列挙が actor_name で dedup されることのテストも追加 (既存テストは actor_name が None だったので見出しだけでも通っていた)。 Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
feat(notify): OS 通知を本家 sw と同じ形式にしてノート本文のダイジェストを載せる
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 25 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. |
github-actions
Bot
added
rust
API surface diff外部アプリ向け API 面 ( Full diffdiff --git a/src-tauri/openapi.json b/src-tauri/openapi.json
index 51e5dcc..68c6716 100644
--- a/src-tauri/openapi.json
+++ b/src-tauri/openapi.json
@@ -6,7 +6,7 @@
"license": {
"name": "MIT"
},
- "version": "1.103.1"
+ "version": "1.103.2"
},
"paths": {
"/api": {
|
Deploying notedeck with
|
| Latest commit: |
3466d12
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://03ef3397.notedeck-d3a.pages.dev |
| Branch Preview URL: | https://develop.notedeck-d3a.pages.dev |
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
変更
🤖 Generated with Claude Code