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: [IOPAE-1048,IOPAE-1049,IOPAE-1051] Adopt the new DS on ServiceDetailsScreen #5677

Merged
merged 12 commits into from
Apr 12, 2024

Conversation

adelloste
Copy link
Member

Short description

This PR updates the service details introducing the footer (special cta + standard ctas), the metadata and the tos&privacy sections.

Details

without ctas with ctas
without-ctas.mp4
with-ctas.mp4
custom flow pn custom flow cgn
cta-special-flow-send.mp4
cta-special-flow-cgn.mp4

List of changes proposed in this pull request

  • added ServiceDetailsTosAndPrivacy component to display Tos&Privacy section
  • added ServiceDetailsMetadata component to display metadata section
  • added ServiceDetailsFooterActions component to display footer: this component allows to display the custom cta for special services or the standard ctas.
  • updated the cta for the pn and cgn flows
  • updated langs

How to test

Using io-dev-api-server, navigate to the services tab and tap a service. Check that the screen is displayed correctly.

@adelloste adelloste requested review from thisisjp and a team as code owners April 9, 2024 19:24
@pagopa-github-bot pagopa-github-bot changed the title [IOPAE-1048,IOPAE-1049,IOPAE-1051] Adopt the new DS on ServiceDetailsScreen chore: [IOPAE-1048,IOPAE-1049,IOPAE-1051] Adopt the new DS on ServiceDetailsScreen Apr 9, 2024
@pagopa-github-bot
Copy link
Collaborator

pagopa-github-bot commented Apr 9, 2024

Warnings
⚠️

Multiple stories with different types are associated with this Pull request.
Only one tag will be added, following the order: feature > bug > chore

Affected stories

  • ⚙️ IOPAE-1048: [APP] Implementare la sezione Termini e Privacy
    subtask of
    • 🌟 IOPAE-1038: [MVP0] Come cittadino voglio vedere pagina di un servizio nazionale/locale con il nuovo DS
  • ⚙️ IOPAE-1049: [APP] Implementare la sezione Contatti e Informazioni
    subtask of
    • 🌟 IOPAE-1038: [MVP0] Come cittadino voglio vedere pagina di un servizio nazionale/locale con il nuovo DS
  • ⚙️ IOPAE-1051: [APP] Implementare le CTA
    subtask of
    • 🌟 IOPAE-1038: [MVP0] Come cittadino voglio vedere pagina di un servizio nazionale/locale con il nuovo DS

Generated by 🚫 dangerJS against f260610

Copy link
Contributor

@Ladirico Ladirico left a comment

Choose a reason for hiding this comment

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

LGTM!

@adelloste adelloste merged commit 9612f85 into master Apr 12, 2024
11 checks passed
@adelloste adelloste deleted the IOPAE-1048-IOPAE-1049-IOPAE-1051 branch April 12, 2024 09:04
adelloste added a commit that referenced this pull request Apr 12, 2024
This pr depends on #5677

## Short description
This PR adds the skeleton for the loading state in
`ServiceDetailsScreen`.

<details open><summary>Details</summary>
<p>

| service details  |
| - |
| <video
src="https://github.com/pagopa/io-app/assets/29163287/befbec7c-57cb-42c7-9067-de424e8d49e7"
width="300"/> |

</p>
</details> 

## List of changes proposed in this pull request
- updated ServiceDetailsScreen in order to display the skeleton

## How to test
Using `io-dev-api-server`, navigate to the services tab and tap a
service. Check that the screen is displayed correctly.
CrisTofani pushed a commit that referenced this pull request Apr 12, 2024
…eDetailsScreen` (#5677)

## Short description
This PR updates the service details introducing the footer (special cta
+ standard ctas), the metadata and the tos&privacy sections.

<details open><summary>Details</summary>
<p>

| without ctas  | with ctas  |
| - | - |
| <video
src="https://github.com/pagopa/io-app/assets/29163287/b6b1e5ff-83d6-4be7-bb56-1bae553bf0b8"
width="300"/> | <video
src="https://github.com/pagopa/io-app/assets/29163287/32e5e218-24bb-4d76-a698-54fdfc675c9b"
width="300"/> |

| custom flow pn  | custom flow cgn  |
| - | - |
| <video
src="https://github.com/pagopa/io-app/assets/29163287/da011192-47a9-42b2-913c-06d8d61222a4"
width="300"/> | <video
src="https://github.com/pagopa/io-app/assets/29163287/6f370405-896c-451c-b533-aa78f2682cd1"
width="300"/> |

</p>
</details> 

## List of changes proposed in this pull request
- added `ServiceDetailsTosAndPrivacy` component to display Tos&Privacy
section
- added `ServiceDetailsMetadata` component to display metadata section
- added `ServiceDetailsFooterActions` component to display footer: this
component allows to display the **custom cta** for special services or
the **standard ctas**.
- updated the cta for the `pn` and `cgn` flows
- updated langs

## How to test
Using `io-dev-api-server`, navigate to the services tab and tap a
service. Check that the screen is displayed correctly.

---------

Co-authored-by: Alice Di Rico <83651704+Ladirico@users.noreply.github.com>
CrisTofani pushed a commit that referenced this pull request Apr 12, 2024
This pr depends on #5677

## Short description
This PR adds the skeleton for the loading state in
`ServiceDetailsScreen`.

<details open><summary>Details</summary>
<p>

| service details  |
| - |
| <video
src="https://github.com/pagopa/io-app/assets/29163287/befbec7c-57cb-42c7-9067-de424e8d49e7"
width="300"/> |

</p>
</details> 

## List of changes proposed in this pull request
- updated ServiceDetailsScreen in order to display the skeleton

## How to test
Using `io-dev-api-server`, navigate to the services tab and tap a
service. Check that the screen is displayed correctly.
CrisTofani pushed a commit that referenced this pull request Apr 12, 2024
…eDetailsScreen` (#5677)

## Short description
This PR updates the service details introducing the footer (special cta
+ standard ctas), the metadata and the tos&privacy sections.

<details open><summary>Details</summary>
<p>

| without ctas  | with ctas  |
| - | - |
| <video
src="https://github.com/pagopa/io-app/assets/29163287/b6b1e5ff-83d6-4be7-bb56-1bae553bf0b8"
width="300"/> | <video
src="https://github.com/pagopa/io-app/assets/29163287/32e5e218-24bb-4d76-a698-54fdfc675c9b"
width="300"/> |

| custom flow pn  | custom flow cgn  |
| - | - |
| <video
src="https://github.com/pagopa/io-app/assets/29163287/da011192-47a9-42b2-913c-06d8d61222a4"
width="300"/> | <video
src="https://github.com/pagopa/io-app/assets/29163287/6f370405-896c-451c-b533-aa78f2682cd1"
width="300"/> |

</p>
</details> 

## List of changes proposed in this pull request
- added `ServiceDetailsTosAndPrivacy` component to display Tos&Privacy
section
- added `ServiceDetailsMetadata` component to display metadata section
- added `ServiceDetailsFooterActions` component to display footer: this
component allows to display the **custom cta** for special services or
the **standard ctas**.
- updated the cta for the `pn` and `cgn` flows
- updated langs

## How to test
Using `io-dev-api-server`, navigate to the services tab and tap a
service. Check that the screen is displayed correctly.

---------

Co-authored-by: Alice Di Rico <83651704+Ladirico@users.noreply.github.com>
CrisTofani pushed a commit that referenced this pull request Apr 12, 2024
This pr depends on #5677

## Short description
This PR adds the skeleton for the loading state in
`ServiceDetailsScreen`.

<details open><summary>Details</summary>
<p>

| service details  |
| - |
| <video
src="https://github.com/pagopa/io-app/assets/29163287/befbec7c-57cb-42c7-9067-de424e8d49e7"
width="300"/> |

</p>
</details> 

## List of changes proposed in this pull request
- updated ServiceDetailsScreen in order to display the skeleton

## How to test
Using `io-dev-api-server`, navigate to the services tab and tap a
service. Check that the screen is displayed correctly.
dhinterlechner added a commit to dhinterlechner/io-app that referenced this pull request Apr 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants