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

Aインスタンスでトゥートを追加ボタンを押してからBインスタンスに変更すると、プレビューとURLが整合的でなくなる #116

Open
sasanquaneuf opened this issue Jan 17, 2020 · 7 comments
Labels
bug Something isn't working

Comments

@sasanquaneuf
Copy link
Contributor

sasanquaneuf commented Jan 17, 2020

<再現手順>
1.インスタンス名= https://qiitadon.com でID=2でプレビュー表示してトゥートを追加
2.インスタンス名= https://mstdn.jp に変更してID=2でプレビュー表示してトゥートを追加

こうすると、別のトゥートが画面上のプレビューに登録されますが、プレビューURLで指定されるインスタンスはmstdn.jpなので、表示が不整合を起こします。

これと本質的に同じ事象としては、
・Aインスタンスでトゥートを登録してから、Bインスタンスのまとめをインポート
・プレビュー表示してからインスタンスを変更して何もせずにトゥートを追加
などがあります。

※あまりそういう事をする人はいない気もしますが、一応

@KEINOS
Copy link
Member

KEINOS commented Jan 17, 2020

実は、他のインスタンスの API レスポンスをみるために、まとめを試してたらそれしちゃいました。

「入力用のテキストボックスを一つにまとめる #37」の対応の PR #115 が完了してから対応するのが楽な気がします。

@yumetodo
Copy link
Contributor

yumetodo commented Jan 17, 2020

これってそういう仕様なんじゃないんですか?同じTooTが2つ出るのが正しいという。 #55 を私はそう解釈しているんですが。

現在の mastogetter の目標は Activity Pub で連携するすべてのインスタンスを単一のスレッド状の status の並びにするものではありません。

@sasanquaneuf
Copy link
Contributor Author

目指す方向性は #55で良いと思いますが、右側のプレビューの内容が書き換わっていないので、プレビューとURLで不整合を起こします。
それがここで挙げている事でした。

あるべき姿は、URLの内容に従って過去プレビューを書き直すか、途中でのインスタンスの変更を禁止するか、ではないかなと思います。

@KEINOS
Copy link
Member

KEINOS commented Jan 17, 2020

同じサーバーの URL しか受け付けないのは仕様ですが、別サーバーの URL を混在させると動いちゃうのはやはり気になります。

UI の改善 #102 に貼り付けたモックの画像( #102 (comment) )で編集中のドメインという ReadOnly のフィールドを入れたのもそこらんの意図がありました。

@KEINOS
Copy link
Member

KEINOS commented Jan 17, 2020

途中でのインスタンスの変更を禁止する

に 👍 。まぁ、一発目のトゥートのドメインから変わったらエラーでいいんじゃないですかね?

@sasanquaneuf
Copy link
Contributor Author

ちなみに、なぜこれを気にし始めたかというと、@useridを含むURLを認めるからでした。
/web/statusの、webログイン後の画面からまとめをつくる場合はほとんど問題ないのですが、@useridのページをまとめづくりに利用する場合、例えばqiitadonから見られるトゥートであっても、インスタンスが別になるため、右側に登録をする前にプレビュー表示した時に違うトゥートになります。
ここまでは、まだ仕様としてもよいのかなと思ったのですが、右側に登録してからインスタンスを変更すると、画面上の見た目は変わらないのにURLでは対象トゥートが変わってしまうので、それはさすがにバグかなと思ったのでした。

@yumetodo
Copy link
Contributor

インスタンス変更というか、UI的には一度入れたら書き換え不能になって、変更にはボタンが必要くらいでよさそう

@blhsrwznrghfzpr blhsrwznrghfzpr added the bug Something isn't working label Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants