Skip to content
This repository was archived by the owner on Feb 22, 2024. It is now read-only.

Hubert / HOTFIX for deposit error message #6785

Merged

Conversation

hubert-deriv
Copy link
Contributor

Initially, the error message was being created through the validation functionality in metatrader config. In this validation it is not possible to inject HTML in a string as it seems that this string gets spit out as text only. So here it is not possible to implement an href in the string. In order to put an href in this error message, I put the error message in the metatrader.jsx file and then made some functionality that displays the error if the user doesn't have enough funds.

@vercel
Copy link

vercel bot commented Sep 14, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/deriv/binary-static/DRUDhY9ZV6B21TQ1XQNYwRQo5aMW
✅ Preview: https://binary-static-git-fork-hubert-deriv-fundtransfererrorbroken.binary.sx

@github-actions
Copy link

github-actions bot commented Sep 14, 2021

A production App ID was automatically generated for this PR. (log)

Click here to copy & paste above information.
- **PR**: [https://github.com/binary-com/binary-static/pull/6785](https://github.com/binary-com/binary-static/pull/6785)
- **URLs**:
    - **w/ App ID + Server**: https://binary-static-git-fork-hubert-deriv-fundtransfererrorbroken.binary.sx/en/logged_inws.html?qa_server=frontend.binaryws.com&app_id=27990
    - **Original**: https://binary-static-git-fork-hubert-deriv-fundtransfererrorbroken.binary.sx/en/logged_inws.html
- **App ID**: `27990`

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
3.1% 3.1% Duplication

@mustofa-binary mustofa-binary merged commit a67713b into binary-com:master Sep 15, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants