misskey icon indicating copy to clipboard operation
misskey copied to clipboard

fix: 非ログイン時にクレデンシャルが必要なページに行くとエラーが出る問題を修正

Open chocolate-pie opened this issue 1 year ago • 4 comments

#10922 の解決

What

非ログイン時にクレデンシャルが必要なページに行くとエラーが出る問題を修正しました。

スクリーンショット 2023-06-08 17 24 33

Why

役に立たない、間違ったエラーメッセージが出るのはよろしくないため

Additional info (optional)

Checklist

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

chocolate-pie avatar Jun 08 '23 08:06 chocolate-pie

Codecov Report

Merging #10973 (03b3f4b) into develop (92d9946) will increase coverage by 0.01%. The diff coverage is 85.71%.

@@             Coverage Diff             @@
##           develop   #10973      +/-   ##
===========================================
+ Coverage    77.37%   77.39%   +0.01%     
===========================================
  Files          908      909       +1     
  Lines        91675    91739      +64     
  Branches      7550     7550              
===========================================
+ Hits         70937    71000      +63     
- Misses       20738    20739       +1     
Impacted Files Coverage Δ
packages/frontend/src/nirax.ts 66.66% <77.77%> (+0.12%) :arrow_up:
packages/frontend/src/components/MkPageWindow.vue 100.00% <100.00%> (ø)
packages/frontend/src/router.ts 96.37% <100.00%> (ø)

... and 1 file with indirect coverage changes

codecov[bot] avatar Jun 08 '23 09:06 codecov[bot]

niraxはMisskeyのドメイン知識とは切り離したい感があるから@/accountとか@/routerに依存したくない気持ちがある

syuilo avatar Jun 09 '23 07:06 syuilo

@syuilo 具体的にどのようにすればいいですか?(@/routerはvueから参照できますが、@/accountはどうすれば...)

chocolate-pie avatar Jun 09 '23 07:06 chocolate-pie

@syuilo 直してみましたが、どうですか?

chocolate-pie avatar Jun 11 '23 03:06 chocolate-pie

👍🏻

syuilo avatar Jul 07 '23 23:07 syuilo