-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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: Use the instance host
instead of the current host
#12226
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #12226 +/- ##
===========================================
+ Coverage 78.85% 78.95% +0.10%
===========================================
Files 945 770 -175
Lines 101959 79039 -22920
Branches 8237 7803 -434
===========================================
- Hits 80397 62409 -17988
+ Misses 21562 16630 -4932 ☔ View full report in Codecov by Sentry. |
このPRによるapi.jsonの差分 差分はこちら |
There is currently an issue where the code implementation is accessing information directly from the API, which can lead to errors when running Test against the frontend like |
非同期的に決定される値をexportするのは良くなさそう |
@syuilo Like this? |
クライアントを開くたびにPOST to metaする実装には反対 |
テストもそのへん(xhr sendしてもサーバーが存在しない)でエラーが出ていそう (だけどそもそも実装方針がよろしくない) |
As I said earlier, the API can't be connected (because it's not even running) |
So far I can only think of this one way to get the instance |
私はそうは思わない 解決策の一つとしては、ビルド時に予めホストを指定すればよい。しかしこれは過去に廃止された、その理由は予めフロントエンドをビルドしてdockerイメージを提供したかったからだ。 (このプルリクエストの目的はapiのドメインとフロントエンドのドメインを分けたいということでいいのですよね?) |
apiサーバーが応答しない場合はテスト以外にも多くあるはずなので、try-catchで適切にエラーを処理する必要がありそう |
フロントエンドとバックエンドのドメインを分けると嬉しいことはわかっているけど、正式にサポートしましたと言うには様々な機能について動作を確認する必要がある(OAuthの動作等) |
What I want is to be able to switch the target domain name of the reverse proxy at any time, for example I can use it directly for a.com and abc.a.com without making any changes. |
Displays the |
What
maybe relate #6724
Why
close #6724
Additional info (optional)
Checklist