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

PTACH エンドポイント修正 #17

Merged

Conversation

gurezo
Copy link
Contributor

@gurezo gurezo commented Mar 24, 2024

PTACH (更新 API )のエンドポイント( URL )の誤りを修正 471b6fd

  • 更新 API の URL は、 id (メモ id )を指定する必要があるので、修正
diff --git a/frontend/src/app/shared/service/api.service.ts b/frontend/src/app/shared/service/api.service.ts
index 8f7d460..10a2e54 100644
--- a/frontend/src/app/shared/service/api.service.ts
+++ b/frontend/src/app/shared/service/api.service.ts
@@ -29,7 +29,7 @@ export class ApiService {
   }

   update(body: Memo) {
-    return this.http.patch(this.apiUrl, body, this.httpOptions);
+    return this.http.patch(this.apiUrl + `/${body.id}`, body, this.httpOptions);
   }

まとめ

  • 上記手順がこのプルリクエストの作業になります。

@ic-lifewood ic-lifewood self-assigned this Mar 24, 2024
@ic-lifewood ic-lifewood merged commit 6b38ae7 into ic-lifewood:main Mar 24, 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.

None yet

2 participants