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

fix(frontend): mCaptchaを使用していてもbotプロテクションに関する警告が消えないのを修正 #14390

Merged
merged 3 commits into from
Aug 11, 2024

Conversation

kakkokari-gtyih
Copy link
Contributor

What

  • mCaptchaを使用していてもbotプロテクションに関する警告が消えないのを修正
  • 型潰し

Why

Fix #14372

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@kakkokari-gtyih kakkokari-gtyih added this to the v2024.8.0 milestone Aug 10, 2024
@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Aug 10, 2024
Copy link

codecov bot commented Aug 10, 2024

Codecov Report

Attention: Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.

Project coverage is 20.25%. Comparing base (37c9d91) to head (1f79074).

Files Patch % Lines
packages/frontend/src/pages/admin/index.vue 0.00% 14 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #14390       +/-   ##
============================================
- Coverage    39.81%   20.25%   -19.57%     
============================================
  Files         1545      723      -822     
  Lines       190495   100283    -90212     
  Branches      3551     1049     -2502     
============================================
- Hits         75839    20308    -55531     
+ Misses      114063    79434    -34629     
+ Partials       593      541       -52     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

if (currentPage.value?.route.name == null && !narrow.value) {
router.push('/admin/overview');
}
});
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

削除した理由は何かしら

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

マウント時のリダイレクトだけで十分なため(実際、設定ページの方はそうなっている)

@syuilo syuilo merged commit 046f243 into misskey-dev:develop Aug 11, 2024
17 checks passed
@syuilo
Copy link
Member

syuilo commented Aug 11, 2024

🙏🏻

@kakkokari-gtyih kakkokari-gtyih deleted the fix-14372 branch August 11, 2024 02:19
LemonDouble pushed a commit to LemonDouble/misskey that referenced this pull request Aug 19, 2024
…#14390)

* fix(frontend): mCaptchaを使用していてもbotプロテクションに関する警告が消えないのを修正

* Update Changelog

* refactor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/frontend Client side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

mCaptcha is not correctly detected
2 participants