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

Misskey Hub内へ飛ばすリンクのリンク切れ解消 #22

Merged
merged 5 commits into from
Nov 15, 2023

Conversation

hilot06
Copy link
Contributor

@hilot06 hilot06 commented Nov 15, 2023

Misskey Hub内へ飛ばすリンクのリンク切れを解消しました(外部リンクについては確認していません)
ほとんどは構造が変わったことによるリンク切れなので差し替えで対応しましたが、差し替えで対応してないやつもあるので一覧にしておきます

  • お気に入りページ(おそらくMisskey Webへ飛ばすリンク?)へのリンク
    →プレーンテキストにした
  • アンケートの解説ページへのリンク
    →新規ファイルを作成しそこへのリンクとした(当該ページについてはWIPです、お待ち下さい)
  • AiScriptの解説ページへのリンク
    →ページがなさそうなのでひとまずTODOとした
  • APIリファレンスのページヘのリンク
    →現在HQで作業中とのことだったのでTODOとした

よろしくお願いします

@hilot06
Copy link
Contributor Author

hilot06 commented Nov 15, 2023

お気に入りページのところ、Misskey Hubの解説ページに飛ばすのかMisskey Webのお気に入りページに飛ばすのか微妙だったのでコミット遡りましたが、Misskey Webに飛ばすリンクっぽいですね…
Misskey Webへのリンクについては #6 に挙がっている misskey-hub#28 にありますし、TODOのほうが良さげでしょうか

@kakkokari-gtyih
Copy link
Collaborator

kakkokari-gtyih commented Nov 15, 2023

お気に入りページのところ、Misskey Hubの解説ページに飛ばすのかMisskey Webのお気に入りページに飛ばすのか微妙だったのでコミット遡りましたが、Misskey Webに飛ばすリンクっぽいですね… Misskey Webへのリンクについては #6 に挙がっている misskey-hub#28 にありますし、TODOのほうが良さげでしょうか

リンク先をTODOとすると結局よく分からん場所に飛ばされてしまうので、リンクを消したうえで直後に(作業中)などを入れていただくのがいいかと

@hilot06
Copy link
Contributor Author

hilot06 commented Nov 15, 2023

たしかにリンク切れ解消させた意味がないですね…
ただ閲覧者にとって(作業中)がかなり唐突になる恐れがある(何の変哲もないプレーンテキストの直後に出るので)ので、コメントアウトで入れてきます

@hilot06
Copy link
Contributor Author

hilot06 commented Nov 15, 2023

そう考えるとほかのTODOたちもプレーンテキスト化→コメントでTODOに変更のほうがええな…

@hilot06
Copy link
Contributor Author

hilot06 commented Nov 15, 2023

AiScriptの解説ページについては、ひとまずAiScriptのリポジトリのREADMEへのリンクに差し替えました(これで事足りそうだったので)
APIリファレンスについてはお気に入りページと同様の修正で

@kakkokari-gtyih kakkokari-gtyih merged commit b2d24ff into misskey-dev:master Nov 15, 2023
@kakkokari-gtyih
Copy link
Collaborator

本当にありがとうございます🙏🙏🙏🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants