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

enhance(games): Gamesのパス・ディレクトリ構造を変更 #13200

Closed

Conversation

kakkokari-gtyih
Copy link
Contributor

What

  • バブルゲーム・リバーシのパスを/games配下に移動(リダイレクトあり)
  • ゲーム系のページvueファイルを/games配下にまとめた

Why

fix #13199

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

@github-actions github-actions bot added the packages/frontend Client side specific issue/PR label Feb 8, 2024
@syuilo
Copy link
Member

syuilo commented Feb 8, 2024

ゲーム系はよく共有されるからURLは短いほど良い気がしている

Copy link

codecov bot commented Feb 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5299d17) 64.35% compared to head (4ba7a35) 77.56%.

Additional details and impacted files
@@             Coverage Diff              @@
##           develop   #13200       +/-   ##
============================================
+ Coverage    64.35%   77.56%   +13.20%     
============================================
  Files          979      185      -794     
  Lines       109398    24757    -84641     
  Branches      4290      463     -3827     
============================================
- Hits         70408    19203    -51205     
+ Misses       38958     5547    -33411     
+ Partials        32        7       -25     

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

@kakkokari-gtyih
Copy link
Contributor Author

うーむ

@tamaina
Copy link
Contributor

tamaina commented Feb 8, 2024

gにでもしとけば(適当)

@kakkokari-gtyih
Copy link
Contributor Author

gにでもしとけば(適当)

リバーシのゲームidと被るので微妙

@tamaina
Copy link
Contributor

tamaina commented Feb 8, 2024

(URLを変える必要はないと思う

@syuilo
Copy link
Member

syuilo commented Feb 8, 2024

既に g は game(対局) の略で使ってるわね(reversi/g/foo)

@syuilo
Copy link
Member

syuilo commented Feb 8, 2024

(URLを変える必要はないと思う

ディレクトリ構造とURLは一対一で対応させときたい

@tamaina
Copy link
Contributor

tamaina commented Feb 8, 2024

じゃあ/gamesの方を非正規URLにするとか(トンチ

@kakkokari-gtyih
Copy link
Contributor Author

kakkokari-gtyih commented Feb 8, 2024

ゲーム系はよく共有されるからURLは短いほど良い気がしている

ならべつに変えなくてもええかの気分

@syuilo
Copy link
Member

syuilo commented Feb 8, 2024

games系としてまとめるならばバックエンドのserviceやmodelもまとめたくなる(実際昔はそうだった)けど、そうするとNestになったりして色々設計が一新された今となっては構造的に不自然になるからバックエンドはそのようにはできない
そうするとフロントエンド側だけまとめることになるけど、それはそれで構造が非対称になるからあまり意味ないかも

🙏

@syuilo syuilo closed this Feb 8, 2024
@kakkokari-gtyih kakkokari-gtyih deleted the enh-13199 branch February 8, 2024 05:03
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.

Misskey Gamesのパスを/games/...配下に移動する
3 participants