Aum/wall 4454/refactor poi poa flow controller #9
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
…count-creation (deriv-com#14711) * fix: reopening of file selector when uploading of files * chore: fix Dropzone for uploading docs on click * chore: minor style fix * fix: fix the reopening issue of dropzone for identity card upload
…icate upload (deriv-com#14703) * fix: issue with datepicker selection and button not disabled upload * chore: fix test cases * chore: removed documentation for p2p-country-list hook * feat: added flow for manual upload error screens * chore: updated content for error * chore: removed test variables * chore: navigate user back to document selection
* fix: fix issues with CFDs password flows - Fix height of success modal in mobile - Fix the title of the modal screens based on selected account - Added WalletErrorMT5Password screen for investor password error - Tried to fix most errors for MT5, MT5Investor, DerivX password issues * fix: fix more issues with footer * fix: sonar issues * chore: add header back for success popup MT5 * fix: header for mt5 investor password reset error and success
…v-com#15320) * chore: remove account-v2 checks from test.yml and chore/build * chore: removed route check * chore: removed prod and local build and test for account-v2 * chore: added jest config to ignore account-v2
) * chore: removed unused code and unused imported variables * chore: reverted passkey file * chore: removed unused code * chore: removed unused code * chore: removed unused code * redeploy: 🚀 to redeploy and trigger build * chore: removed unwanted code * chore: removed unused and duplicated file --------- Co-authored-by: Muhammad Fasih Ali Naqvi <[email protected]>
…nimum withdrawal amount, the withdrawal limit shown below the amount field displays in a Max-Min pattern (deriv-com#15265) * feat(wallets): 🚑 add balanceLessThanMinWithdrawalLimit cehck to crypto withdraw * test(wallets): 🧪 add test case
…d />` (deriv-com#15317) * test: add test file for WalletClipboard * test: add test case for timeout * refactor: `fireEvent -> userEvent` * refactor: a few adjustments * refactor: a small change * fix: inconsistency * refactor: better strucure
…-poi-poa (deriv-com#15336) * fix: disabling of next button if file is missing in poi or poa flows * fix: fix types and ci
Co-authored-by: DerivFE <[email protected]>
…discrepancies (deriv-com#15345) * fix: poa and personal details validations and discrepancies * fix: updated city validator regexp
deriv-com#15298) * fix: allow overflow * refactor: only horizontal overflow * chore: update quill-icons * refactor: remove `shouldAllowOverflow` thing; change tooltip alignment * refactor: typo
…iv-com#15347) * fix: account creation success message fix * fix: applied comments
…or (deriv-com#15278) * feat: added CryptoLimitAgeVerified error screen for WithdrawalCrypto * fix: fix TS error * feat: added countLimitMessageFn to handle daily count transfer limit * feat: added daily count message for demo wallets
…ntroducing Traders Hub's logged out version (deriv-com#14740) * chore: initial setup * chore: test case fix * chore: fixing test cases * chore: fixing test cases * chore: binary link fix * chore: add root route / * feat: started * feat: intermediate result * feat: intermediate result 2 * feat: add 1st test case * chore: review fix * chore: fixing test case * chore: redirecting trading button to dtrader * fix: test case * fix: trigger * feat: change hardcoded dtrader route to routes.trade * feat: add trustpilot widget * refactor: extract the code to separate components * feat: add getLandingCompany to client store * feat: create trustpilot helper function * feat: add some icons for banner and trustpilot * feat: intermediate result * feat: complete widget and banner * feat: refactor the code a little bit * feat: complete with options and multipliers * feat: change trustpilot data type * feat: changed content for CFDs part * feat: renamed some components * feat: change 3 headers * fix: add mock for header tests * feat: complete changes for header * feat: change trust pilot widget data * feat: separate styles * feat: add some styles * fix: remove exclamation mark * feat: make legacy TH route as root * fix: implement sonalcloud suggestions * fix: repair tests * feat: get back changes for content because it will be done by card trah-3251 * feat: create separate components for option and cfds titles and description * feat: remove waiting for landing company and show just eu and non-eu versions * feat: change routes to legacy th * feat: change routes to legacy th 2 * fix: repair tests due to route changes * feat: add exact to the trades_hub route * feat: add position absolute to the image * feat: add redirection from old traders hub route to the new one * fix: same route for wallets and TH * fix: change route order to fix old TH route redirect * feat: add redirection to dtrader * fix: wallet_transfer route in shared * chore: code clean-up * chore: more code clean-up * fix: move wallets check out of appstore Co-authored-by: Aum <[email protected]> * chore: reset file to base branch * fix: compare account route height and hover shadow clipping * chore: remove unused import * fix: repair wallet tests * fix: add logged_in_app_platform and logged_out_app_platform * feat: show content for incognito * chore: refactor routes.ts for better readability * fix: insufficient balance modal button routing * fix: routing to contract page giving 404 * feat: change options & multipliers to options * feat: update mobile header --------- Co-authored-by: ahmadtaimoor-deriv <[email protected]> Co-authored-by: nijil-deriv <[email protected]> Co-authored-by: Aum <[email protected]>
…-update (deriv-com#15346) * fix: WithdrawalNoBalance shown if balance changes to 0 * fix: options trading icons for TH in mobile view * fix: some test cases * chore: remove unused import * fix: applied suggestion
Co-authored-by: DerivFE <[email protected]>
Co-authored-by: DerivFE <[email protected]>
Co-authored-by: DerivFE <[email protected]>
* chore: updated border radius for mt5 modal * chore: update comment suggestion
…tementTransaction to TS (deriv-com#15105) * test: MarketSymbolIconRow + migrate formatStatementTransaction to TS * test: MarketSymbolIconRow finalized
* chore: initial setup * chore: test case fix * chore: fixing test cases * chore: fixing test cases * chore: binary link fix * chore: add root route / * chore: review fix * chore: fixing test case * chore: redirecting trading button to dtrader * fix: test case * fix: trigger * chore: stylelint
…en in the overlay / non-root wallets page (deriv-com#15231) * fix: onboarding btn in wallets * refactor: apply suggestion * fix: close modal pre tour guide * fix: the console error * refactor: remove unused usages * fix: useEffect dep array * fix: 🐛 show onboarding for wallet responsive * refactor: correct route usages --------- Co-authored-by: mitra <[email protected]>
* chore: re-added overflow auto * chore: update ModalStepWrapper with disableScroll props * chore: updated dropdown height to be more appropriate on desktop view
…MT5 for Demo Account (deriv-com#15378) * chore: update market type for demo account * chore: move values within fn
Co-authored-by: DerivFE <[email protected]>
Co-authored-by: DerivFE <[email protected]>
* Update smoketests_manual.yml Modified record key var * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests_manual.yml * Update smoketests.yml * Update smoketests_manual.yml * Update .github/workflows/smoketests.yml Co-authored-by: balakrishna-deriv <[email protected]> * Update smoketests_manual.yml Added Wallets * Update smoketests.yml * Update smoketests_manual.yml --------- Co-authored-by: balakrishna-deriv <[email protected]>
…ng in glitch when trying to place order again after market rate change (deriv-com#15814) * fix: glitch showing fluctuation modal on confirm, use error message from BE for slippage * fix: removed commented out code * chore: wrap create new order with localize
… on P2P buy/sell tab are not aligned in desktop and responsive mode (deriv-com#15832) * fix: reverted back the style changes in dropdown.scss * chore: empty commit
* feat: add check for wallets and dont hide landsacpe blocker for this kind of accounts * feat: use has_wallet from client store instead
Co-authored-by: DerivFE <[email protected]>
* Removed inputs Removed inputs as they are a security risk in open repo. They're, not necessary for the test anyway. * Update smoketests_manual.yml * Update smoketests_manual.yml * Update .github/workflows/smoketests_manual.yml Co-authored-by: Ali(Ako) Hosseini <[email protected]> * Update .github/workflows/smoketests_manual.yml Co-authored-by: Ali(Ako) Hosseini <[email protected]> * Update .github/workflows/smoketests_manual.yml Co-authored-by: Ali(Ako) Hosseini <[email protected]> --------- Co-authored-by: Ali(Ako) Hosseini <[email protected]>
…ion (deriv-com#15815) * feat: add useDevice hook instead of is_mobile from the store * feat: changed a lot of stuff * fix: tests for personal-details * feat: implement suggestions * feat: change js breakpoints, add styles for tablet screens and fix tests
* feat: 🔥 add accumulators in deriv-bot * feat: 🚀 Fixed accumulator edit take profit and bot stuck issue * fix: ⚡ fixed edit take profit for the accumulator type contract for bots * fix: ⚡ show apporpriate message on server errir * fix: 🐛 the take profit block should be disabled if there is no amount * fix: 🐛 disable accumulator for the quick strategy form * feat: ✨ added accumulator blocks in the blocks menu * style: 💄 Improved position of trade type on summary panel * fix: ✅ on change currency it should reflect - accumulator * fix: ✅ should not show tick dots for accumulators
Co-authored-by: DerivFE <[email protected]>
…riv-com#15195) * feat: added crypto priority withdrawal feature for wallets * fix: removed unnecessary click events in lable in Wallet checkbox * fix: failing test cases for WalletCryptoProvider * fix: fixed the error handling for crypto address field * fix: display of amount received * fix: changed the units in scss file of walletCheckbox from fix to relative * refactor: moved withdraw checkbox and priorty info into a seprate component in cashier * fix: fixed failing test case in withdraw store * fix: fixed the info icon in priority withdrwawl checkbox * fix: fixed failing test case * feat: changed the crypto_estimations to a subscribe call * fix: unsubscribe the crypto_estimations call and fixed failing test cases * fix: fixed failing test cases * fix: replaced checkbox icon with quil icon * feat: added transaction fee in transaction detail of crypto withdrawal * fix: resolved failing type error * fix: disable eslint camel case for for one line in CryptoTrasaction * fix: added todo to remove transfer fee from transaction type in TCryptoTransaction * fix: changed eslint-disable only for one specific line in CryptoTransaction * fix: implemented review comment on the text to display transaction fee * feat: added amount received in crypto withdrwawal receipt * fix: the logic to show the transaction feee in the withdraw-crypto-receipt * fix: fixed failing test cases related to withdrawal-crypto * fix: fix the display condition for transfer fee in withdraw crypto receipt * feat: added crypto-priority-withdrawal behind a feature flag * fix: fixed the failing test cases in withdraw crypto form after adding useGrowthhookIsOn hook
* Update smoketests.yml The set env vars step wasn't resolving the label collection correctly. * Update smoketests.yml
Co-authored-by: DerivFE <[email protected]>
* Farhan/Lock Dtrader Tablet View on Landscape Mode (deriv-com#14781) * chore: lock landscape mode on tablet view * chore: only show tablet view on real tablet * fix: notification * chore: change implementation * fix: change root * refactor: comments * fix: comments * fix: height * fix: rtl language * refactor: remove extra property * fix: positions modal height based on figma * chore: add loader before rotating the screen * fix: failing test * chore: add timeout before hiding loader * style: move styles * chore: another ref for loader timeout cleanup * fix: hide footer for dtrader * feat: added teamp breakpoints * feat: enable tablet view on accounts * feat: enabled table view for financial assessment * feat: tablet design for personal details * chore: update css files * feat: enable table view for financial assesment * feat: enabled tablet view * feat: removed unwanted code * feat: updated with logical property * feat: removd ui store and added devices * feat: updated nav condition * fix: removed usage of is_mobile, changed mixin name with latest, footer * fix: padding of content and footer btn * DTRA / Kate / FEQ-2212 / Hide footer navigation links popovers (deriv-com#15176) * fix: add check for desktop * refactor: add custom check for tablet os * refactor: revert changes from endpoint note and server time * refactor: device detection * fix: style and update function for detection tablet os * chore: remove empty space * chore: remove used type * DTRA / Kate / FEQ-2213 + FEQ-2205 / Made popover optional (deriv-com#15197) * fix: add check for purchase button popover for tablet * fix: make popover optional for header * refactor: add condition for wallets header * [DTRA] Farhan/Dtrader Tablet Subtasks (deriv-com#15123) * chore: lock landscape mode on tablet view * chore: only show tablet view on real tablet * fix: notification * chore: change implementation * fix: change root * refactor: comments * fix: comments * fix: height * fix: rtl language * refactor: remove extra property * fix: positions modal height based on figma * chore: add loader before rotating the screen * fix: failing test * chore: add timeout before hiding loader * style: move styles * chore: another ref for loader timeout cleanup * fix: hide footer for dtrader * chore: update os detect * fix: failing tests * fix: height issue * fix: android device issue and toolbar widgets * fix: reports height * fix: test failing * fix: position drawer on ios * refactor: remove unused prop * fix: route for contract --------- Co-authored-by: balakrishna-deriv <[email protected]> * chore: update @deriv/deriv-charts to 2.1.16 (deriv-com#15212) Co-authored-by: balakrishna-deriv <[email protected]> * style: initial style changes * feat: tablet view changes for languages * feat: removed unwanted new line * [DTRA] Farhan/Fix test failing (deriv-com#15216) * chore: lock landscape mode on tablet view * chore: only show tablet view on real tablet * fix: notification * chore: change implementation * fix: change root * refactor: comments * fix: comments * fix: height * fix: rtl language * refactor: remove extra property * fix: positions modal height based on figma * chore: add loader before rotating the screen * fix: failing test * chore: add timeout before hiding loader * style: move styles * chore: another ref for loader timeout cleanup * fix: hide footer for dtrader * chore: update os detect * fix: failing tests * fix: height issue * fix: android device issue and toolbar widgets * fix: reports height * fix: test failing * fix: position drawer on ios * refactor: remove unused prop * fix: route for contract * fix: failing test and ts * fix: miseed ts errors --------- Co-authored-by: balakrishna-deriv <[email protected]> * feat: get changes back and just delete mixin for tablet (deriv-com#15182) * [FEQ] Sergei / FEQ-2200, FEQ-2201 / Add a real MT5 account, Identity verification, Choose a jurisdiction popup for tablet (deriv-com#15205) * feat: change styles for mt5 jurisdiction modal * feat: change styles for adding a real mt5 account modal * feat: change more styles * fix: add mock for useDevice hook from deriv-com/ui * style: update self-exclusion deign tablet * chore: update mixin names * feat: enabled tablet view for trading assessment * style: tablet design api token * [DTRA] Farhan/FEQ-2208/Hide indicators and drawing tools toolbar widgets on tablet (deriv-com#15224) * chore: lock landscape mode on tablet view * chore: only show tablet view on real tablet * fix: notification * chore: change implementation * fix: change root * refactor: comments * fix: comments * fix: height * fix: rtl language * refactor: remove extra property * fix: positions modal height based on figma * chore: add loader before rotating the screen * fix: failing test * chore: add timeout before hiding loader * style: move styles * chore: another ref for loader timeout cleanup * fix: hide footer for dtrader * chore: update os detect * fix: failing tests * fix: height issue * fix: android device issue and toolbar widgets * fix: reports height * fix: test failing * fix: position drawer on ios * refactor: remove unused prop * fix: route for contract * fix: failing test and ts * fix: miseed ts errors * fix: hide indicators and drawing tools for tablet view * fix: test case * fix: height reports --------- Co-authored-by: balakrishna-deriv <[email protected]> * style: tablet design acnt limits * feat: enabled poa for tablet view * feat: removed unused code * feat: used not desktop breakpoint * fix: lof=gical property for css * fix: css * feat: updated device hook * style: tablet view, modal fix * style: tablet view changes for various POO status components, common-css * [DTRA] Farhan/FEQ-2205/Header notification icon and account settings links are not working as expected (deriv-com#15275) * fix: hide popover on traders hub header * fix: account switcher width * fix: comments * feat: enabled tablet view for proof of income section * style: tablet view for 2FA * fix: build_issue * chore: email-pswd-section * feat: change mixins and add isTablet for datepicker (deriv-com#15349) * feat: change 2 desktop mixins (deriv-com#15362) * fix: new button fixes * fix: fixed containers * fix: fixed comments1 * fix: fixed bug in desktop view * fix: fixed comments 2 * fix: fixed comments 5 * fix: fixed comments 6 * fix: fixed padding and changed px to rem * fix: removed lines * fix: fixed code * feat: added missing code which already merged * chore: clean code * fix: remove unused css * fix: personal details section * [DTRA] Farhan/Tablet view subtasks (deriv-com#15376) * fix: subtasks 2 * fix: failing tests * fix: fixed scrolling issue for tablet and mobile * chore: remove old is mobile and is desktop and added new * fix: removed unwanted code * fix: fixed footer width * test: fix testcases for useDevice hook * test: update api-token testcase * test: update api-token testcase * fix: fixed width and scroll issue * test: update api-token testcase * chore: added new break points and removed old * fix: update testcases * fix: close-acnt modal * fix: address comments * fix: address comments * [DTRA] Farhan/FEQ-2280/tablet subtasks (deriv-com#15404) * fix: subtasks 2 * fix: failing tests * fix: positions toggle icon * refactor: digit breakpoint * fix: drawer moving when coming back from reports * fix: tablet tooltip * fix: hide download widget for tablet * fix: report open position footer * fix: double sell button * fix: digit container * fix: failing tests * fix: tests --------- Co-authored-by: Farhan Ahmad Nurzi <[email protected]> * chore: removed drag drop for tablet view * fix: use new breakpoints r passkeys components * fix: POI test cases * fix: POI test cases2 * fix: POI test cases3 * fix: clean up and test cases in POI * fix: POI clean up * fix: style for tradingassessment * fix: cleanup 3 * fix: style for tradingassessment * fix: removed height * fix: removed typoerror * fix: removed code2 * chore: remove desktop/mobile wrappers * chore: remove desktop/mobile wrappers * fix: rtl changes * style: update closing acnt * chore: update prop name * chore: update prop name * style: close account margin * fix: added missing code * fix: added missing code2 * chore: resovled conflicts * fix: replaced Desktop() with useDevice hook * fix: replaced Desktop() with useDevice hook2 * fix: replaced Desktop() with useDevice hook3 * fix: replaced Desktop() with useDevice hook4 * fix: replaced Desktop() with useDevice hook5 * fix: replaced Desktop() with useDevice hook5 * style: empty commit to resolve conflicts * FEQ-2313 / Kate / Android tablet- Long code message content is showing in smaller size (deriv-com#15474) * fix: css for link * fix: css * chore: get back line height * refactor: add tablet os check and revert style changes * Fix tablet view issues (deriv-com#15469) * fix: jest test trader * fix: alignment issues * fix: test cases * fix: contract type dialog height (deriv-com#15476) * fix: added isDesktop to missing places2 * chore: fixed modal height issue and semicolon issue * chore: removed extra semicolon * fix: added isDesktop to missing places3 * fix: personal details error * fix: added isDesktop to missing places4 * Ahmad/Fix Strike Icon color on firefox (deriv-com#15481) * chore: strike icon * fix: revert all * fix: tablet issues (deriv-com#15497) * fix: padding top for sections * chore: fixed semocolon and scrollbar issue * fix: fieldset max width * fix: close acnt btn alignment * fix: fixed scroll and padding in POI * fix: close acnt btn alignment * fix: px to rem fix * fix: testcases * chore: remove unused hook * chore: updated with parent branch and resolved conflicts * Revert "Fasih/ Updated with parent branch and resolved conflicts" * fix: poi scroll screen issue * [FEX] Sergei / FEQ - 2310, 2311, 2319, 2320, 2344 / Tablet view issues #3 (deriv-com#15522) * fix: change isDisabledLandscapeBlockerRoute because traders hub route now is the root route * feat: make logged out version of TH accesable for tablet view * feat: add styles for mt5 password modal * feat: fix notifications styles * fix: fixed build issue * fix: added test cases for passkeys * Ahmad/Strike Icon Svg fix on Firefox (deriv-com#15578) * chore: strike icon * fix: revert all * chore: add important * fix: poi fixes1 * fix: fixed testcases * fix: fixed testcases2 * fix: poi fixes2 * Ahmad/ Fiix Strike Icon in firefox and update Deriv UI Package (deriv-com#15609) * chore: strike icon * fix: revert all * chore: add important * fix: update packages and remove temp fix * fix: address comments * fix: selfexclusion testcase * fix: trading-assessment-user-observer-remove * fix: api and connected apps * fix: css fixes * fix: icon1 * chore: resolved comments and added fragment * fix: self exclusion and css * fix: endpoint page * fix: creation banner and adding mt5 account (deriv-com#15659) * Revert "fix: creation banner and adding mt5 account (deriv-com#15659)" (deriv-com#15661) This reverts commit ef2dddb. * fix: api token layout * [FEQ] Sergei / FEQ - 2374, 2375, 2376 / Tablet view issues 4 (2) (deriv-com#15666) * fix: creation banner and adding mt5 account * feat: add deriv-com/ui to account package json * fix: add react/jsx-runtime to webpack resolvers because i have added deriv-com/ui to package json * chore: fixed width of tray on mobile * chore: addressed comments * chore: removed duplicated import * fix: border radius api card * fix: btn aligmnent desktop * fix: pull invalid date fix * chore: back btn margin fix * [FEQ] Sergei / FEQ - 2376 / "Get real account" button for DIEL (deriv-com#15690) * feat: tablet view suggestion * feat: implement new design for tablet * fix: fix passkeys and coverall issue * fix: build fail * fix: build fail * fix: fix import --------- Co-authored-by: Farhan Ahmad Nurzi <[email protected]> Co-authored-by: balakrishna-binary <[email protected]> Co-authored-by: Muhammad Fasih Ali Naqvi <[email protected]> Co-authored-by: utkarsha-deriv <[email protected]> Co-authored-by: kate-deriv <[email protected]> Co-authored-by: balakrishna-deriv <[email protected]> Co-authored-by: utkarsha-deriv <[email protected]> Co-authored-by: DerivFE <[email protected]> Co-authored-by: balakrishna-deriv <[email protected]> Co-authored-by: Sergei Baranovski <[email protected]> Co-authored-by: fasihali-deriv <[email protected]> Co-authored-by: Dhruv Neb <[email protected]> Co-authored-by: Farhan Ahmad Nurzi <[email protected]> Co-authored-by: Muhammad Fasih Ali Naqvi <muhammadfasih@Muhammad-Fasihs-Mac-X7KDC4NKQ0-Standard-No-NextDNS.local> Co-authored-by: Akmal Djumakhodjaev <[email protected]> Co-authored-by: ahmadtaimoor-deriv <[email protected]> Co-authored-by: Likhith Kolayari <[email protected]> Co-authored-by: Habib Deriv <[email protected]>
Co-authored-by: DerivFE <[email protected]>
Co-authored-by: DerivFE <[email protected]>
Co-authored-by: DerivFE <[email protected]>
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:
This PR is to refactor converted IDVDocumentUpload to IDVService.
Screenshots:
Please provide some screenshots of the change.