Suisin/merge with current master #16876
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…reation (binary-com#16811) * feat: hide one time deposit for mf clients in real account creation * chore: remove unused variables * chore: fix test cases * chore: re-word the test case --------- Co-authored-by: nijil-deriv <nijil@deriv.com>
…ary-com#16786) * feat: tusdt notice banner and notification for wallets/th * fix: applied comments * fix: subtask * fix: subtask * fix: deposit crypto max width
…-Web-terminal-CTA-button (binary-com#16671) * chore: updated trade modal ui in TH and WALL * chore: remove unused imports * chore: remove remaining WalletButton components
…ll in minimize view is scrollable vertically (binary-com#16808) * chore: add fixed height * refactor: remove unused files * refactor: remove more unused files
…tion Sheet (binary-com#16787) * chore: done * fix: fix * fix: test case * fix: adding in mockStore * chore: icon * chore: fix * fix: test case * fix: changing snakcase * chore: review comments * fix: review fix for signup * chore: fix * chore: fix
… currency (binary-com#16472) * feat: add check for currency * fix: add check for optional function from props
…d-clients (binary-com#16720) * fix: resubmission for fully authenticated clients * fix: added check for previous attempt status * refactor: make is_poi_required code readable
…om#16564) * chore: replace wallettext with text component * chore: revert back to previous type * chore: replace text with loader and add missed translations * chore: add translations * revert stylelint changes * chore: revert more stylelint changes * ci: fix failing tests
Co-authored-by: DerivFE <fe-access@regentmarkets.com>
Co-authored-by: DerivFE <fe-access@regentmarkets.com>
binary-com#16577) * [BOT]maryia/BOT-1981/feat: Add a "What's new" section to the dashboard to promote Accumulators (binary-com#16369) * feat: announcement components * refactor: announcement config, make accounce dialog reusable * feat: make redirect and show block accumulator * feat: show indicator updated upon clicking on announcements * refactor: separate an announcement component * feat: add saving to local storage, refactor the code * chore: fix sonarcloud issues * chore: fix sonarcloud issues(2) * chore: fix sonarcloud issues(3) * feat: redirect to default strategy with accumulator upon clicking "try now" button * feat: redirect to default strategy with accumulator upon clicking "try now" button * refactor: click on announce button * feat: add video for accumulator feature on tutorial page * chore: simplify access to the context of the DOM element * test: announcement component * test: announcement-dialog component * test: announcements component * chore: add cursor for the button * chore: remove temporary other announcements except of accumulator feature * refactor: styles and remove unused import * chore: use global colors * chore: rename names of the tests * chore: change colors as per global colors * refactor: minor things * chore: remove UserGuide component and test * chore: remove dependency * chore: add @deriv/quill-icons to package.json * chore: replace .then() to await waitFor * chore: remove some code * chore: change color to global color that support dark and light mode * chore: refactor implementation, rename variables * refactor: announcements component, configs and helper functions * fix: sonar cloud issues * fix: test cases * fix: The icon and texts are not aligned as per design * fix: The cta size is different compared to design * fix: text indent * refactor: announcement-dialog * fix: styles announcement-dialog * style: improve * refactor: improvement accourding to comments (binary-com#16603) * feat: adapt Notifications component (binary-com#16608) * refactor: improvement accourding to comments * feat: adapt Notifications component * chore: rename prop of notifications * chore: rename prop of notifications (binary-com#16658) * chore: renaming child of content from content to content_item * chore: split .notifications_wrapper as per SASS convention * chore: make the same everywhere React.useState( way * chore: rename stored_notifications to read_announcements_map * chore: rename temp_data to updated_local_storage_data * chore: rename getButtonAction to performButtonAction * chore: update the version of deriv-com/ui * fix: the issue after update the version in dtrader * fix: provide the notifications component with new mandatory prop loadMoreFunction * chore: package-lock.json * chore: update version of deriv-com/ui * fix: remove loadMoreFunction prop * refactor: method to calc amount active announce, state for it * chore: add wrapper arrow function for handleRedirect method * fix: Unable to see the pop up on tab view * chore: add missed dependency, add eslint comments * fix: modal width on tablet * refactor: improve styles as per design on mobile version * chore: update the date for accumulator announce * chore: merge master(part2)
* fix: server side pdf fix * fix: code smell * fix: code smell * fix: switch case * fix: snake case
…merge_with_current_master
suisin-deriv
requested review from
adrienne-deriv,
thisyahlen-deriv,
wojciech-deriv,
ali-hosseini-deriv,
amina-deriv,
balakrishna-deriv,
farrah-deriv,
hirad-deriv and
maryia-deriv
as code owners
suisin-deriv
requested review from
matin-deriv,
nijil-deriv,
sandeep-deriv,
yashim-deriv,
aizad-deriv,
heorhi-deriv and
vinu-deriv
as code owners
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
amina-deriv
merged commit 9a109a9
into
binary-com:suisin/phone_number_verification_uat_branch
3 of 4 checks passed
|
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-suisin-deriv-suisin-mergewithcurrentmaster.binary.sx/ |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Changes:
Please provide a summary of the change.
Screenshots:
Please provide some screenshots of the change.