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

NSFWを常に表示にする設定でも年齢確認処理を行う前は隠す #251

Merged
merged 5 commits into from
Jul 29, 2024

Conversation

kozakura913
Copy link
Collaborator

#245 の欠陥修正

Checklist

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

@kozakura913
Copy link
Collaborator Author

TLに流し込む時に隠すか処理してるみたいで年齢確認処理してもすでに読み込まれた部分は変化しない

Copy link

codecov bot commented Jul 29, 2024

Codecov Report

Attention: Patch coverage is 68.75000% with 10 lines in your changes missing coverage. Please review.

Project coverage is 78.18%. Comparing base (e1b4508) to head (7297c2b).

Files Patch % Lines
packages/frontend/src/components/MkMediaImage.vue 50.00% 6 Missing ⚠️
packages/frontend/src/scripts/check-r18.ts 33.33% 4 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##           develop     #251       +/-   ##
============================================
+ Coverage    64.42%   78.18%   +13.76%     
============================================
  Files         1072      198      -874     
  Lines       132151    29367   -102784     
  Branches      5652      475     -5177     
============================================
- Hits         85140    22962    -62178     
+ Misses       46978     6399    -40579     
+ Partials        33        6       -27     

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

@kozakura913 kozakura913 merged commit 61916d8 into yojo-art:develop Jul 29, 2024
8 of 10 checks passed
@kozakura913 kozakura913 deleted the check_r18_any_nsfw_settings branch July 29, 2024 09:48
Copy link

sonarcloud bot commented Jul 29, 2024

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

Successfully merging this pull request may close these issues.

2 participants