[NO-TASK] Update recommendation FAQ texts #757
Merged
Conversation
This file contains hidden or 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
davidleomay
approved these changes
Dec 2, 2025
TaprootFreak
added a commit
that referenced
this pull request
Dec 3, 2025
* [NO-TASK] Fix KuCoin Pay typo (#755) * Hide assets with zero balance in Safe portfolio (#756) * [NO-TASK] Update recommendation FAQ texts (#757) * [NO-TASK] Update recommendation FAQ texts * [NO-TASK] Add additional FAQ questions to recommendation screen * [NO-TASK] Safe source asset auto-select * [NO-TASK] FAQ refactoring * [DEV-4484] Mail login prefill, invitations w/o mail * Add PDF Download Address Report button to account screen (#762) * [DEV-4484] Recommendation code parameter * Add PDF export functionality for Safe portfolio (#761) * Add PDF download button to Safe portfolio page - Add PDF icon button in the portfolio header (top right) - Add translations for 'Download PDF' in DE, FR, IT * Move PDF button to 'Total portfolio value' row and fix z-index - Position PDF icon next to 'Total portfolio value' text - Add z-20 to ensure hover effect works over chart overlay * Add PDF date selection modal - Add modal with date picker when clicking PDF button - User can select a date for the portfolio report - Add 'Generate PDF' button (functionality to be implemented) - Add translations for modal texts in DE, FR, IT * Add PDF export functionality for Safe portfolio - Add downloadPdf function to safe.hook.ts for fetching balance PDF - Connect PDF modal in safe.screen.tsx to actual API endpoint - PDF endpoint: GET /balance/pdf with address, blockchain, currency, date params * Fix PDF export to use authenticated API call - Use useApi call() instead of fetch() for proper authentication - Handle base64 encoded PDF response from API - Add language parameter to PDF request * Refactor: use downloadPdfFromString utility for PDF export * Use custody/pdf endpoint for Safe PDF export - Switch from balance/pdf to custody/pdf endpoint - Remove address/blockchain params (API handles aggregation) - Update filename to DFX_Safe_Balance_Report * Improve Safe PDF modal to match account screen - Add loading state and error handling with UI feedback - Add Close (X) button and Cancel button - Use StyledInput type="date" instead of DatePicker - Change date param from Date to string (YYYY-MM-DD) * Remove unused 'Generate PDF' translation * Add missing translations for PDF Download Address Report in screens/home * Remove language parameter from PDF download API now uses language from user data instead of parameter * Fix Trezor manifest missing appName property (#760) * Fix Trezor manifest missing appName property Add required appName field to TrezorConnect manifest configuration * Update @trezor/connect-web to 9.6.4 Update package-lock.json to use the latest version which requires the appName property in the manifest configuration * Add @types/lodash to fix TypeScript build The @trezor/connect-web 9.6.4 update introduced a transitive dependency on lodash that requires TypeScript type definitions. * PDF export refactoring * [DEV-4512] add tradingNotAllowed tx error --------- Co-authored-by: David May <david.leo.may@gmail.com> Co-authored-by: Yannick1712 <52333989+Yannick1712@users.noreply.github.com>
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.
Summary
Test plan