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

Akmal / fix: wrong duration unit for digit contracts #16378

Conversation

akmal-deriv
Copy link
Contributor

Changes:

This fix corrects the issue of displaying the wrong duration unit for digit contracts, ensuring that the duration information is accurate and consistent.

  • Correct Duration Unit: Update the logic to display the correct duration unit for digit contracts, matching the expected values.
  • User Interface: Adjust the user interface to reflect the corrected duration unit, providing users with accurate and clear information.

Copy link

vercel bot commented Aug 6, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
deriv-app ✅ Ready (Inspect) Visit Preview Aug 19, 2024 5:57am

Copy link
Contributor

github-actions bot commented Aug 6, 2024

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

Click here to copy & paste above information.
- **PR**: [https://github.com/binary-com/deriv-app/pull/16378](https://github.com/binary-com/deriv-app/pull/16378)
- **URLs**:
    - **w/ App ID + Server**: https://deriv-app-git-fork-akmal-deriv-b-digits-unit.binary.sx?qa_server=red.derivws.com&app_id=23938
    - **Original**: https://deriv-app-git-fork-akmal-deriv-b-digits-unit.binary.sx
- **App ID**: `23938`

Copy link
Contributor

github-actions bot commented Aug 6, 2024

🚨 Lighthouse report for the changes in this PR:

Category Score
🔺 Performance 33
🟧 Accessibility 70
🟧 Best practices 83
🟧 SEO 77
🟧 PWA 78

Lighthouse ran with https://deriv-app-git-fork-akmal-deriv-b-digits-unit.binary.sx/

Copy link

sonarcloud bot commented Aug 19, 2024

@vinu-deriv vinu-deriv merged commit 3eea540 into binary-com:position-and-contract-details-fixes Aug 21, 2024
4 checks passed
vinu-deriv pushed a commit that referenced this pull request Aug 27, 2024
* Ahmad/DTRA-1408/Fix Vanilla Contract in Contract Details V2 (#16018)

* fix: decimal point

* fix: currency in payoutperpoint

* fix: vanillas

* fix: only vanilla

* fix: test coverage

* Ahmad/DTRA-1405/Multipliers fix on Contract Details V2 (#16017)

* fix: multiplier

* fix: testcase

* fix: fix

* fix: deal cancel

* chore: add DC

* chore: suggestion

* chore:fix

* fix: test case fix

* Ahmad/DTRA-1401/DTRA-1402/Add currency in payout per point and decimal point in take profit (#16012)

* fix: decimal point

* fix: currency in payoutperpoint

* fix: vanillas

* fix: test case and comments

* chore: review fix

* fix: indent

---------

Co-authored-by: balakrishna-deriv <56330681+balakrishna-deriv@users.noreply.github.com>

* fix: bugs with resetting filter and icon for hight low (#16003)

Co-authored-by: balakrishna-deriv <56330681+balakrishna-deriv@users.noreply.github.com>

* Akmal / fix: pass currency across closed positions and contract details (#16161)

* fix: pass currency across closed positions and contract details

* fix: add a comment

* [DTRA]/Ahmad/DTRA-1490/Deal Cancellation Status Fix Trader-V2 (#16121)

* fix: cancellation active, executed and expired

* chore: revert last commit

* DTRA-1569 / Kate / Filter resetting issue (#16359)

* fix: resetting filter issue

* refactor: add date clear on unmount

* refactor: tests

* fix: tests

* fix: console error

* Akmal / fix: wrong duration unit for digit contracts (#16378)

* fix: wrong duration unit for digit contracts

* fix: test case

* Akmal / fix: take profit & stop loss decimals (#16195)

* fix: remove decimals from take profit

* fix: apply fixes from quill library

* fix: apply dynamic decimal places based on currency

* feat: update quill-ui version

---------

Co-authored-by: ahmadtaimoor-deriv <129935294+ahmadtaimoor-deriv@users.noreply.github.com>
Co-authored-by: kate-deriv <121025168+kate-deriv@users.noreply.github.com>
Co-authored-by: Akmal Djumakhodjaev <akmal@binary.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants