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

Fix: Enable API_SERVER injection in netlify build #49

Merged
merged 4 commits into from
Jul 26, 2018

Conversation

coord-e
Copy link
Contributor

@coord-e coord-e commented Jul 25, 2018

Step 1: 再現済み環境

  • OS: Linux coordiserver 4.17.0-1-amd64#1 SMP Debian 4.17.8-1 (2018-07-20) x86_64 GNU/Linux
  • devenv: 90569b09bf78e0d93eff82eb8aff941774adc111
  • frontend: 2aae84b
  • backend: 0f77f92764e0ef1627b05c1070bcbae621d3291a

Step 2: 変更内容

  • REACT_APP_STAGING_BUILDがtruthyならば、NODE_ENVdevelopmentではなくてもAPI_SERVERをCookie経由で変更することを許可する
  • Netlifyでのビルド時にREACT_APP_STAGING_BUILDをセットする
  • partial fix for netlifyがproduction buildになっている #48

Step 2: 検証手順

再現のための手順:

(検証前にyarn global add serve)

  1. yarn build
  2. serve buildで開くので、API_SERVERをCookieからつけてもリクエスト先が変わっていないことを確認する
export REACT_APP_STAGINIG_BUILD=1
yarn build
  1. serve buildで開くので、API_SERVERをCookieからつけるとリクエスト先が変わっていることを確認する

修正前の挙動:

  • yarn buildするとAPI_SERVERをCookieからつけてもリクエスト先が変わらない

修正後の挙動:

  • REACT_APP_STAGINIG_BUILD=1なら、yarn buildでもAPI_SERVERをCookieからつけてリクエスト先が変わる

Step 3: 影響範囲

  • ない

@coord-e coord-e added bug Something isn't working partial Partial fix labels Jul 25, 2018
@coord-e coord-e added this to the v0.1 milestone Jul 25, 2018
@coord-e coord-e requested a review from a team July 25, 2018 06:53
@coord-e
Copy link
Contributor Author

coord-e commented Jul 25, 2018

netlifyデプロイがデフォルトでdevenvのバックエンドを指すような変更もしました
別のPR/backendのdevelopを使いたい場合はAPI_SERVERでやればいいので

@coord-e coord-e added the Immediate Immediate action is requested label Jul 26, 2018
@anharu2394
Copy link
Member

うまくいきませんでした。なぜでしょう。

@coord-e
Copy link
Contributor Author

coord-e commented Jul 26, 2018

検証手順を変更しました。

@coord-e
Copy link
Contributor Author

coord-e commented Jul 26, 2018

ローカルでこの手順を踏むことはないし、netlifyの方では動作が確認できるので目的は達成されたと思っているのですが、どうでしょう

Copy link
Member

@anharu2394 anharu2394 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

netlifyでは検証ずみだそうです。

@coord-e coord-e merged commit 7831729 into develop Jul 26, 2018
@coord-e coord-e deleted the bugfix/48-netlify-build branch July 26, 2018 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Immediate Immediate action is requested partial Partial fix
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants