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

enhance(backend): 絵文字の連合でライセンス欄を相互にやり取りするように #14109

Closed
wants to merge 5 commits into from
Closed

enhance(backend): 絵文字の連合でライセンス欄を相互にやり取りするように #14109

wants to merge 5 commits into from

Conversation

KisaragiEffective
Copy link
Collaborator

@KisaragiEffective KisaragiEffective commented Jul 1, 2024

What

address #10859

Why

issueの通り

Additional info (optional)

  • テストはかけてない (現状書けない?) ので動作確認が必要。
  • 新規に検知した絵文字のみのはず

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@KisaragiEffective KisaragiEffective added the 🌌Federation The Federation/ActivityPub feature label Jul 1, 2024
@KisaragiEffective KisaragiEffective self-assigned this Jul 1, 2024
@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Jul 1, 2024
Copy link
Contributor

github-actions bot commented Jul 1, 2024

このPRによるapi.jsonの差分

差分はこちら

Get diff files from Workflow Page

Copy link

codecov bot commented Jul 1, 2024

Codecov Report

Attention: Patch coverage is 41.66667% with 7 lines in your changes missing coverage. Please review.

Project coverage is 40.16%. Comparing base (070f0e7) to head (21e6eb0).

Files Patch % Lines
...ckend/src/core/activitypub/models/ApNoteService.ts 0.00% 4 Missing ⚠️
.../backend/src/core/activitypub/ApRendererService.ts 0.00% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #14109       +/-   ##
============================================
+ Coverage    20.46%   40.16%   +19.70%     
============================================
  Files          701     1521      +820     
  Lines        98653   188564    +89911     
  Branches      1025     2663     +1638     
============================================
+ Hits         20185    75733    +55548     
- Misses       77947   112290    +34343     
- Partials       521      541       +20     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@KisaragiEffective
Copy link
Collaborator Author

KisaragiEffective commented Jul 1, 2024

misskey-hubに上げるために 39daef9 時点のセマンティクスを明文化:

  1. IApEmoji_misskey_license という追加のフィールドを持っていても良い。フィールドが存在する場合、その内容はMisskey::ApLicenseMetadata である必要がある。
  2. Misskey::ApLicenseMetadata は次のフィールドを持つ。下にリストアップされていないフィールドがペイロードの内部にあってもよいが、そのようなフィールドは単に無視され、全く効果を持たない。
    • liecnse: MiEmoji.license に対応するフィールド。自由記述欄のため、機械可読ではない。

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Jul 2, 2024

  • ネームスペースを宣言する必要がありそう
  • misskey hub の ns.mdに追記する必要がありそう

あとプロパティを新規に生やすお作法がこれで正しいかは不明

// Misskey拡張。後方互換性のためにoptional。
// 将来の拡張性を考慮してobjectにしている
_misskey_license?: {
freeText: string | null;
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
freeText: string | null;
// nullの場合、リモートサーバーにおいて明文のライセンスが設定されていないことを示す。
// 非nullの場合、リモートサーバーに置いてライセンスがその内容で明文化されていることを示す。
freeText: string | null;

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

あああspace

@@ -229,6 +229,11 @@ export interface IApEmoji extends IObject {
type: 'Emoji';
name: string;
updated: string;
// Misskey拡張。後方互換性のためにoptional。
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Suggested change
// Misskey拡張。後方互換性のためにoptional
// Misskey拡張。後方互換性及び他ソフトウェアとの互換性のためにoptional

@KisaragiEffective
Copy link
Collaborator Author

@KisaragiEffective
Copy link
Collaborator Author

機運が満ちていないのでclose

@KisaragiEffective KisaragiEffective deleted the feat/backend/prefer-newly-added-misskey-extension-license-field-to-ap-emoji branch July 20, 2024 09:15
lynlynlynx added a commit to lynlynlynx/misskey that referenced this pull request Dec 28, 2024
enhance(backend): 絵文字の連合でライセンス欄を相互にやり取りするように (misskey-dev#14109)
@github-actions github-actions bot mentioned this pull request Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌌Federation The Federation/ActivityPub feature packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants