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

chore: [IOBP-757] Apply info copy component into transaction details screen #6362

Merged

Conversation

LeleDallas
Copy link
Contributor

@LeleDallas LeleDallas commented Nov 4, 2024

Short description

This pull request is adding clipboard functionality to certain items in transaction details screen.

List of changes proposed in this pull request

  • Replace ListItemInfo component with ListItemInfoCopy

How to test

  • Navigate to the Pagamenti section
  • Select any receipt
  • Tap on its details
  • Verify that the Tax Code, Codice Avviso, IUV and Oggetto del pagamento fields are copyable
  • Confirm that the copied values match the original information accurately

Preview

Screenshot 2024-11-05 at 08 58 53 Screenshot 2024-11-05 at 08 59 08

@pagopa-github-bot pagopa-github-bot changed the title refactor: [IOBP-757] apply info copy component into transaction details screen chore: [IOBP-757] Apply info copy component into transaction details screen Nov 4, 2024
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Nov 4, 2024

Affected stories

  • ⚙️ IOBP-757: [FE] Implementazione cta per copy in schermata PAYMENT_TRANSACTION_BIZ_EVENTS_CART_ITEM_DETAILS
    subtask of
    • 🌟 IOBP-750: Possibilità di copiare il codice avviso/CF Ente dalla schermata di dettaglio transazione (PAYMENT_TRANSACTION_BIZ_EVENTS_CART_ITEM_DETAILS)

Generated by 🚫 dangerJS against 9433aae

Copy link

codecov bot commented Nov 4, 2024

Codecov Report

Attention: Patch coverage is 0% with 7 lines in your changes missing coverage. Please review.

Project coverage is 47.01%. Comparing base (4f204b4) to head (9433aae).
Report is 670 commits behind head on master.

Files with missing lines Patch % Lines
...mentsTransactionBizEventsCartItemDetailsScreen.tsx 0.00% 4 Missing ⚠️
...on/screens/PaymentsTransactionOperationDetails.tsx 0.00% 3 Missing ⚠️
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #6362      +/-   ##
==========================================
- Coverage   48.42%   47.01%   -1.42%     
==========================================
  Files        1488     1817     +329     
  Lines       31617    36938    +5321     
  Branches     7669     8838    +1169     
==========================================
+ Hits        15311    17365    +2054     
- Misses      16238    19507    +3269     
+ Partials       68       66       -2     
Files with missing lines Coverage Δ
...on/screens/PaymentsTransactionOperationDetails.tsx 8.00% <0.00%> (-0.70%) ⬇️
...mentsTransactionBizEventsCartItemDetailsScreen.tsx 4.16% <0.00%> (ø)

... and 1440 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa65e18...9433aae. Read the comment docs.

Copy link
Contributor

@Hantex9 Hantex9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@LeleDallas LeleDallas merged commit f620db4 into master Nov 5, 2024
13 checks passed
@LeleDallas LeleDallas deleted the IOBP-757-copy-action-in-BIZ-EVENTS-CART-ITEM-DETAILS branch November 5, 2024 08:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
IO-Bonus e pagamenti IO - Bonus e pagamenti
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants