Skip to content

feat: add settings menu#849

Merged
pietro-tota merged 17 commits intomainfrom
CHK-4708-service-settings-section
Feb 11, 2026
Merged

feat: add settings menu#849
pietro-tota merged 17 commits intomainfrom
CHK-4708-service-settings-section

Conversation

@pietro-tota
Copy link
Contributor

@pietro-tota pietro-tota commented Feb 2, 2026

Note for reviewers: development have been split in multiple PRs in order to make review process easier.
This PR is the first one where all other PRs will be merged in order to have a releasable branch

List of Changes

Add settings menu section to allow ECs to express/revoke consent to use RTP service

Motivation and Context

Add settings menu section to allow ECs to express/revoke consent to use RTP service

How Has This Been Tested?

Test locally using dev env as target env + jest tests (See test pr)

Screenshots (if appropriate):

Screenshot 2026-02-05 alle 11 15 43 Screenshot 2026-02-05 alle 11 15 53 Screenshot 2026-02-05 alle 11 16 01 Screenshot 2026-02-05 alle 11 18 30 Screenshot 2026-02-05 alle 11 18 52 Screenshot 2026-02-05 alle 11 18 57 Screenshot 2026-02-05 alle 11 19 02 Screenshot 2026-02-05 alle 11 19 47

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as
    expected)

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

1 similar comment
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

1 similar comment
@github-actions
Copy link

github-actions bot commented Feb 2, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 3, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@github-actions
Copy link

github-actions bot commented Feb 4, 2026

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@pietro-tota pietro-tota marked this pull request as ready for review February 4, 2026 15:47
@pietro-tota pietro-tota requested a review from svariant as a code owner February 4, 2026 15:47
@pietro-tota pietro-tota requested review from a team and gioelemella as code owners February 4, 2026 15:47
@pietro-tota pietro-tota marked this pull request as draft February 4, 2026 15:53
@github-actions
Copy link

github-actions bot commented Feb 5, 2026

This pull request does not contain a valid label. Please add one of the following labels: [major, minor, patch, patch, skip]

@pietro-tota pietro-tota marked this pull request as ready for review February 6, 2026 08:21
CianoDanilo
CianoDanilo previously approved these changes Feb 9, 2026
@CianoDanilo
Copy link
Contributor

@pietro-tota Please add the required labels

* empty

* feat: add popup + use trans for links

* feat: popup menus

* chore: dummy jwt token

* feat: add info alert

* chore: add translations

* feat: add loader

* chore: add TODO message

* feat: add state changer + mock update action

* feat: status chip calculation (#851)

* feat: status chip calculation

* chore: code refactoring

* chore: code review

* chore: service enabling data

* chore: remove edit contacts button

* chore: activation date calculation

* fix: env variable name

* chore: code optimization

reduce to one modal instead of duplicating the same one
make use of react component instead of function invocation

* [CHK-4708] feat: service settings add PUT consent api invocation (#852)

* feat(consent-api): add save consent API invocation on button click

* revert local env file

* feat(consent-api): fix ConsentEnum and Date usage

* feat(consent-api): lint

* feat(consent-api): update error title and description

* chore: merge modification + code refactoring

* [CHK-4708] feat: service settings add get api invocation (#853)

* feat(consent-api): add GET service consents API invocation

* feat(consent-api): update error title and description

* Chk 4708 unit tests (#855)

* feat(wip): test

* chore: restore components

* chore: add missing import + early return

* chore: add error message

* feat: wip tests

* feat: test + page refactoring to help in test

* fix: tests

* chore: remove comment

* chore: lint

* fix: key stiles

* trigger CR pipeline

* chore: update can i use browser list

* chore: add test for errors

* [CHK-4708] feat(integration-test): add service settings page integration tests (#857)

* feat(integration-test): add service settings page integration tests

* feat(integration-test): remove only

* feat(integration-test): add service settings page integration tests

* feat(integration-test): remove only

* feat(integration-test): refactor code

* feat(integration-test): refactor code

* feat: additional permission check to show components (#856)

* feat: additional permission check to show components

* chore: restore dummy value

* chore: enable service modification keys only for admins

* chore: code refactoring

* chore: fix test

* fix: test import

* feat: add tests

* trigger CR pipeline

* feat: add tests for institution service

* chore: remove unused import

* chore: add tests for coverage

---------

Co-authored-by: CianoDanilo <danilo.ciano@pagopa.it>

---------

Co-authored-by: Pietro Tota <115724836+pietro-tota@users.noreply.github.com>

---------

Co-authored-by: Pietro Tota <115724836+pietro-tota@users.noreply.github.com>
Co-authored-by: pietro.tota <pietro.tota@pagopa.it>

---------

Co-authored-by: CianoDanilo <danilo.ciano@pagopa.it>

---------

Co-authored-by: CianoDanilo <danilo.ciano@pagopa.it>
@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

CianoDanilo
CianoDanilo previously approved these changes Feb 10, 2026
Copy link
Contributor

@CianoDanilo CianoDanilo left a comment

Choose a reason for hiding this comment

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

🔥

@github-actions
Copy link

This PR exceeds the recommended size of 400 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions
Copy link

Azure Static Web Apps: Your stage site is ready! Visit it here: https://icy-island-050778e03-849.westeurope.5.azurestaticapps.net

@sonarqubecloud
Copy link

@pietro-tota pietro-tota merged commit 0836f94 into main Feb 11, 2026
14 checks passed
@github-actions github-actions bot deleted the CHK-4708-service-settings-section branch February 11, 2026 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants