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

Editing notes #8362

Closed
ThatOneCalculator opened this issue Feb 27, 2022 · 28 comments
Closed

Editing notes #8362

ThatOneCalculator opened this issue Feb 27, 2022 · 28 comments
Labels
✨Feature This adds/improves/enhances a feature

Comments

@ThatOneCalculator
Copy link
Contributor

Summary

As of 17 days ago, Mastodon supports editing notes (and viewing the edit history). I think this is a major feature that would be a really big enhancement for Misskey, and shouldn't be too hard to implement.

@ThatOneCalculator ThatOneCalculator added the ✨Feature This adds/improves/enhances a feature label Feb 27, 2022
@Johann150
Copy link
Contributor

Johann150 commented Feb 27, 2022

First of all I think that the ability to edit a note is a bad idea. I also don't understand how you get the idea that it would be "not too hard to implement".

In my opinion, the current delete & redraft system works fine. It can fix everything that you could fix by editing a note, while avoiding the many pitfalls that come with editing an existing note.

Furthermore Misskey is not Mastodon, so just because they do it doesn't mean we have to do it too.

@syuilo
Copy link
Member

syuilo commented Feb 27, 2022

First of all I think that the ability to edit a note is a bad idea. I also don't understand how you get the idea that it would be "not too hard to implement".

In my opinion, the current delete & redraft system works fine. It can fix everything that you could fix by editing a note, while avoiding the many pitfalls that come with editing an existing note.

Furthermore Misskey is not Mastodon, so just because they do it doesn't mean we have to do it too.

It is true that there are not many occasions when people want to edit a Note, and the current "delete and edit" is not much of a problem, so it would be a low priority.

@nullobsi
Copy link
Contributor

I'm not sure about the implementation details of Mastodon's editing feature. Perhaps a new issue should be opened for at least federating/receiving post edits from Mastodon?

@tamaina
Copy link
Contributor

tamaina commented Feb 27, 2022

How does mastodon implement federation of this feature?
@mei23

@Johann150
Copy link
Contributor

Johann150 commented Feb 27, 2022

How does mastodon implement federation of this feature?

mastodon/mastodon#16697 implemented receiving updates from the federation and showing what was changed. Then mastodon/mastodon#17320 later allowed changing some attributes of a note via Mastodon's API. It is federated as an Update activity here:
https://github.com/mastodon/mastodon/blob/cb2e198d89dfb86a81fff8e11eac531b658e9ef2/app/workers/activitypub/status_update_distribution_worker.rb#L19-L27

@tamaina
Copy link
Contributor

tamaina commented Feb 27, 2022

I have one more question: Does Misskey support Update activity from remote?

@Johann150
Copy link
Contributor

Johann150 commented Feb 27, 2022

Looks like only Actors and Questions can be updated currently:

if (isActor(object)) {
await updatePerson(actor.uri!, resolver, object);
return `ok: Person updated`;
} else if (getApType(object) === 'Question') {
await updateQuestion(object).catch(e => console.log(e));
return `ok: Question updated`;
} else {
return `skip: Unknown type: ${getApType(object)}`;
}

@tamaina
Copy link
Contributor

tamaina commented Feb 27, 2022

I think Misskey should support updation of note text from remote immediately.

@Johann150
Copy link
Contributor

I think Misskey should support updation of note text from remote immediately.

👀👇

Perhaps a new issue should be opened for at least federating/receiving post edits from Mastodon?

Since this issue is about being able to edit notes when you are on Misskey.

@tusooa
Copy link

tusooa commented Jul 1, 2022

I implemented editing in Pleroma: https://git.pleroma.social/pleroma/pleroma/-/merge_requests/3678

@ThatOneCalculator

This comment was marked as off-topic.

@Johann150
Copy link
Contributor

I think the correct issue for that is #8364.

@tamaina
Copy link
Contributor

tamaina commented Nov 23, 2022

個人的な意見

  • Misskeyにノート編集(notes/update, Note Update)を実装する必要はない、編集すると何に対してリアクションしたかが変わってくるので削除して編集の方が適切だと思う
  • Mastodonなどの他のソフトで行われた変更を受信した際は編集を適用しリアルタイムに反映する必要がある→ Recieve and Apply Mastodon Note Edits #8364
  • Misskeyでリモートの編集履歴を保持する必要はない、編集履歴はオリジナルを参照すれば良い(Mastodonは嘘をつかないだろう)
    編集済みフラグを保持し、何かしらの編集がされたことを表示するだけ

@rinsuki
Copy link
Contributor

rinsuki commented Nov 23, 2022

mastodon.socialとかならさておき別に悪意あるサーバーなら嘘つきまくれる

@kato83
Copy link

kato83 commented Jan 3, 2023

削除して編集でも良いとは思うのですが、Misskey をマイクロブログ(ブログ)として扱い人にとっては
コンテンツを編集してしまうと投稿時に URL が変わってしまうのが Google 検索だったり他のサイト等からの
自然流入の観点的には喜べるものでは無いのかなと思ったりしました。
(issueでそんな観点が挙がってないのでそのような用途ではあまり使用されていないのかもしれないですが……)

SNS として使う分には上でのやりとりの通り問題ないかなと。

@acid-chicken
Copy link
Member

ノートが編集できた方が自然なユースケースが出てきた (See #3217)

@suoko
Copy link

suoko commented Jun 20, 2023

It woukd be a really nice feature

@ThatOneCalculator
Copy link
Contributor Author

Inbound and outbound editing has already been implemented and refined in Calckey, and Misskey is free to reference our implementation.

@Vavency
Copy link

Vavency commented Sep 25, 2023

It could be worth taking a look at sharkey for this due to it being a recent fork that has note edits.

@syuilo
Copy link
Member

syuilo commented Sep 29, 2023

done

@syuilo syuilo closed this as completed Sep 29, 2023
@ThatOneCalculator
Copy link
Contributor Author

This issue should be reopened. As far as I understand, editing has only been implemented for local posts (and newly fetching previously edited remote posts)

@syuilo
Copy link
Member

syuilo commented Sep 30, 2023

#11929

@syuilo
Copy link
Member

syuilo commented Oct 4, 2023

後からメンションとかハッシュタグ追加された場合の対応などが面倒ということがわかった

@syuilo
Copy link
Member

syuilo commented Oct 4, 2023

廃止したい

@syuilo
Copy link
Member

syuilo commented Oct 4, 2023

実装コストに見合う価値がなさそうだし廃止するかしら

@tamaina
Copy link
Contributor

tamaina commented Oct 4, 2023

廃止に賛成

削除して編集でいい

@acid-chicken
Copy link
Member

少なくとも今の実装を据え置くよりは廃止した方が良い

@saschanaz
Copy link
Member

削除して編集でいい

マストドンからの編集への対応は必要かなと

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature This adds/improves/enhances a feature
Projects
None yet
Development

No branches or pull requests