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

フロントエンドのエンドポイントの設定方法を axios に統一する #1375

Closed
4 tasks
1nu1taichi opened this issue Jun 24, 2024 · 1 comment · Fixed by #1382
Closed
4 tasks
Assignees
Labels
target: ガイド/AP開発手順 ドキュメントのガイド/アプリケーション開発手順に関係がある target: Azure AD B2C Auth Azure AD B2C認証の要件別サンプルに関係がある target: Dressca サンプルアプリケーションDresscaに関係がある
Milestone

Comments

@1nu1taichi
Copy link
Contributor

1nu1taichi commented Jun 24, 2024

概要

OpenAPI generator の更新により、エンドポイントの設定が apiClient だけでなく axios でもできるようになり、axiosの設定値が最も優先順位が高くなった。
OpenAPITools/openapi-generator#17296

そのため OpenAPI generator の有無にかかわらず axios でエンドポイントの設定ができるようになるため、axios に統一したい。

詳細

以下のように修正する。

/** api-client の共通の Configuration があればここに定義します。 */
function createConfig(): apiClient.Configuration {
  const config = new apiClient.Configuration({
-    basePath: import.meta.env.VITE_AXIOS_BASE_ENDPOINT_ORIGIN,
  });
  return config;
}

/** axios の共通の設定があればここに定義します。 */
const axiosInstance = axios.create({
+  baseURL: import.meta.env.VITE_AXIOS_BASE_ENDPOINT_ORIGIN,
  headers: {
    'Content-Type': 'application/json',
  },
  withCredentials: true,
});

影響範囲

完了条件

  • エンドポイントの設定が axios で行われている
    • Dressca
    • AzureADB2C
  • エンドポイントの設定に関するドキュメントが正しく更新されている
@1nu1taichi 1nu1taichi added ドキュメント改善 target: Dressca サンプルアプリケーションDresscaに関係がある target: ガイド/AP開発手順 ドキュメントのガイド/アプリケーション開発手順に関係がある target: Azure AD B2C Auth Azure AD B2C認証の要件別サンプルに関係がある labels Jun 24, 2024
@tsuna-can-se tsuna-can-se added this to the v0.10 milestone Jun 24, 2024
@1nu1taichi
Copy link
Contributor Author

1nu1taichi commented Jun 26, 2024

axios: baseURL で設定した際、URLに正常に反映されることを確認(dev, build→previewの両方で確認)
↓ エンドポイントに https://localhost:3000 を指定
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
target: ガイド/AP開発手順 ドキュメントのガイド/アプリケーション開発手順に関係がある target: Azure AD B2C Auth Azure AD B2C認証の要件別サンプルに関係がある target: Dressca サンプルアプリケーションDresscaに関係がある
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants