-
Notifications
You must be signed in to change notification settings - Fork 303
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
[Refactoring] shontzu/CFDS-1435/use-landing-company-hooks-for-cTrader-and-DxTrade #11608
[Refactoring] shontzu/CFDS-1435/use-landing-company-hooks-for-cTrader-and-DxTrade #11608
Conversation
…country of residence
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
A production App ID was automatically generated for this PR. (log)
Click here to copy & paste above information.
|
🚨 Lighthouse report for the changes in this PR:
Lighthouse ran with https://deriv-app-git-fork-shontzu-deriv-shontzu-cfds-1435use-la-1bac54.binary.sx/ |
Co-authored-by: Muhammad Hamza <120543468+hamza-deriv@users.noreply.github.com>
🚀 Smoke test run (1) passed successfully! |
🚀 Smoke test run (2) passed successfully! |
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
❌ Smoke test run (1) failed. See logs for details: Visit Action |
❌ Smoke test run (2) failed. See logs for details: Visit Action |
🚀 Smoke test run (2) passed successfully! |
🚀 Smoke test run (1) passed successfully! |
create a hook using
landing_company
endpoint to get available cTrader and dxTrade accountscurrent behaviour
CFDs_restricted_countries
andfinancial_restricted_countries
to filter for available cTrader and dxTrade accounts:expected behaviour
landing_company
hook to check if ctrader and dxtrade are allowed in client's country of residencedxtrade_all_company
will be in the response, else, it is not in the responsectrader
will be in the response and the jurisdiction will be svg. Else,ctrader
will be in the response but the jurisdiction will be empty string (' ') orundefined