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.

Sign up for GitHub

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

Jump to bottom

embed: ユーザーノート一覧のURLはusernameを使用する #14532

Open
1 task done
kakkokari-gtyih opened this issue Sep 9, 2024 · 8 comments
Open
1 task done

embed: ユーザーノート一覧のURLはusernameを使用する #14532

kakkokari-gtyih opened this issue Sep 9, 2024 · 8 comments
Labels
✨Feature This adds/improves/enhances a feature

Comments

@kakkokari-gtyih
Copy link
Contributor

kakkokari-gtyih commented Sep 9, 2024 •
edited
Loading

Summary

ユーザーノートタイムラインの埋め込みURLを /embed/user-timeline/@foobar にする

Purpose

Do you want to implement this feature yourself?

  • Yes, I will implement this by myself and send a pull request
@kakkokari-gtyih kakkokari-gtyih added the ✨Feature This adds/improves/enhances a feature label Sep 9, 2024
@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Sep 9, 2024 •
edited
Loading

いったんこれを言った後けっきょくユーザーIDベースで実装を行ったが、やっぱりこっちのほうがいい気がしたので再び提案

@kakkokari-gtyih
Copy link
Contributor Author

意見を募集

@syuilo
Copy link
Member

syuilo commented Sep 9, 2024 •
edited
Loading

URLはこっち側が用意するもので、基本的に手動で作られることを想定していない

@anatawa12
Copy link
Member

静的サイトジェネレータ系やワードプレス等CMS系でユーザのurlから作りたいという需要はあると思う

@kakkokari-gtyih kakkokari-gtyih added this to the v2024.9.0 milestone Sep 9, 2024
@syuilo
Copy link
Member

syuilo commented Sep 10, 2024

ID使ってほしいわね

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Sep 10, 2024 •
edited
Loading

静的サイトジェネレータ系やワードプレス等CMS系でユーザのurlから作りたいという需要はあると思う

今のままだとハンドル→ユーザーIDで一度APIを叩く手間が発生しそう

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Sep 10, 2024 •
edited
Loading

静的サイトジェネレータ系やワードプレス等CMS系でユーザのurlから作りたいという需要はあると思う

今のままだとハンドル→ユーザーIDで一度APIを叩く手間が発生しそう

oEmbedなら別にいいかもしれないけど、動的に生成する場合、標準的な実装では対応できない（or 埋め込むユーザー側にどこかしらから自身のユーザーIDをコピペしてもらう必要がある）可能性がある

@kakkokari-gtyih kakkokari-gtyih removed this from the v2024.9.0 milestone Sep 19, 2024
@kakkokari-gtyih
Copy link
Contributor Author

次のバージョンのリリースまでに仕様を固めておいたほうがいいかなと思っていたけど、usernameを対応させるだけなら別にあとからでもできそうなので一旦マイルストーン（入れたいリスト）から外した

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

No branches or pull requests
3 participants
@syuilo @anatawa12 @kakkokari-gtyih