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

検索でhttps://から始まるものを検索すると自動的に照会になる #12832

Closed
1 task done
GrapeApple0 opened this issue Dec 28, 2023 · 3 comments
Closed
1 task done
Labels
⚠️bug? This might be a bug

Comments

@GrapeApple0
Copy link
Sponsor Contributor

GrapeApple0 commented Dec 28, 2023

💡 Summary

(仕様な気もしますがバグで立てます)
検索でhttps://から始まるもの(https://google.comなど)を検索すると自動的に照会になってしまいます

🥰 Expected Behavior

自動的に照会にならない/またはモードを設定できる

🤬 Actual Behavior

自動的に照会になる

📝 Steps to Reproduce

  1. 検索でhttps://から始まるものを検索する

💻 Frontend Environment

* Model and OS of the device(s): HP Z240
* Browser: Vivaldi 6.5.3206.48 (Stable channel) (64-bit) 
* Server URL: misskey.04.si
* Misskey: 2023.12.2-pie-3.0.9

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service:
* Misskey:
* Node:
* PostgreSQL:
* Redis:
* OS and Architecture:

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@GrapeApple0 GrapeApple0 added the ⚠️bug? This might be a bug label Dec 28, 2023
@caipira113
Copy link
Contributor

caipira113 commented Dec 28, 2023

Screenshot_20231228_144615_Chrome Dev
💡 If it's added, I'd like to see it integrated with the existing search options.

  • I realised that it could be annoying if the option was reset every time I searched.

@zyoshoka
Copy link
Contributor

既に閉じられてしまっていますが #10839 と同じですね。

まず #12736 を直して、ap/show へのリクエストで NO_SUCH_OBJECT が返ってきた場合に普通のクエリとして検索するようにするのが良さそうな気がしています。

@Sayamame-beans
Copy link
Member

#13858 で照会を行うかどうか確認を入れるようにしました

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants