-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[WIP] ノート埋め込み機能 #10491
[WIP] ノート埋め込み機能 #10491
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #10491 +/- ##
============================================
+ Coverage 79.79% 99.37% +19.58%
============================================
Files 956 4 -952
Lines 108800 4476 -104324
Branches 8372 58 -8314
============================================
- Hits 86814 4448 -82366
+ Misses 21986 28 -21958 ☔ View full report in Codecov by Sentry. |
埋め込みにおいてMisskey Webを呼び出すのは、クリックジャッキング対策が困難であることから現実的ではないと思います。 |
3b8f8f2
to
ae22b5b
Compare
現在pugによるSSRを行う方向で実装中です |
…notes-embed" This reverts commit a5fd0e1.
|
Fix #1714
What
Why
現状の懸念点
→ Misskey Webと分離(共通部分は最低限にして、本体が読み込まれないように調整。APIリクエストは純粋な
fetch
を使用し、トークンの取得を回避)(埋め込む側ではCDNからコードを引っ張るようにしている)
→ URLパラメータでカスタマイズできるようにした
のこり
MkSparkle
に相当する部分の実装Additional info (optional)
埋め込み
埋め込みコードコピー
生成される埋め込みコード
uuidが生成されて被りができないようにしてある
APIリクエストが発生する場面
init.ts
でのAPIリクエスト(純粋なfetch
オブジェクト使用)Checklist