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
/chat のURLを変更 #40
/chat のURLを変更 #40
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #40 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 12 16 +4
Lines 39 56 +17
Branches 2 5 +3
=========================================
+ Hits 39 56 +17
☔ View full report in Codecov by Sentry. |
@@ -26,7 +26,7 @@ describe('src/api/client/fetchCatMessage.ts fetchCatMessage TestCases', () => { | |||
|
|||
it('should be able to fetch a CatMessage', async () => { | |||
const fetchedResponse = await fetchCatMessage({ | |||
catName: 'moko', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
バックエンド側では cat_id
なので合わせて変更した。
issueURL
#38
この PR で対応する範囲 / この PR で対応しない範囲
#38 のDoneの定義を満たす実装はこのPRで全て完了させる。
Storybook の URL、 スクリーンショット
UI変更は行っていないのでなし。
変更点概要
/chat
のURLを/chat/moko
のようにねこのIDを入れる形に変更。これは今後、ねこのIDが増えた際にURLを変えたくないので、今のうちに対応したかったので対応した。
レビュアーに重点的にチェックして欲しい点
特になし
補足情報
インラインコメントに記載。