Skip to content

Update Wallet Metadata and EC#1092

Open
giadas wants to merge 19 commits intoitalia:eudiwfrom
giadas:iss-1076-wpmetadata
Open

Update Wallet Metadata and EC#1092
giadas wants to merge 19 commits intoitalia:eudiwfrom
giadas:iss-1076-wpmetadata

Conversation

@giadas
Copy link
Copy Markdown
Collaborator

@giadas giadas commented Apr 17, 2026 •
edited
Loading

This PR resolves

The files modified are 6:

  • remote-flow.rst (en and it)
  • wallet-solution-metadata.rst (en and it)
  • test-plans-wallet-provider.rst (en and it)

the others result from the switch from italia:versione-corrente to italia:eudiw

peppelinux and others added 17 commits April 9, 2026 14:55
@peppelinux
specs: updated rp md choices href and version
3b34031
@peppelinux
Update README to remove Semantic Versioning section
b02e4f8 
Removed mention of Semantic Versioning from the README.
@peppelinux
logging policy: added Authentic Sources
75db3f6
@peppelinux
terms: better definition of LoA
6aee403
@pcerqua-ipzs
fix: example versions
25892c8
@giadas
chore: update wallet metadata and EC
302d979
@m-basili
Merge branch 'versione-corrente' into lts
e0866e3
@m-basili
feat: resolution issues italia#1058, italia#1059 and italia#1060
c98a14f
@m-basili
fix: Credential version
1f2cb9b
@peppelinux
Merge pull request italia#1082 from italia/rp-choices
0733568 
specs: updated rp md choices href and version
@peppelinux
Apply suggestions from code review
3f8d086 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@peppelinux
Apply suggestions from code review
5bafd80 
Co-authored-by: Giuseppe De Marco <demarcog83@gmail.com>
@peppelinux
Apply suggestions from code review
709686c 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Giuseppe De Marco <demarcog83@gmail.com>
@peppelinux
Merge pull request italia#1083 from italia/log-as
29898c8 
logging policy: added Authentic Sources
@peppelinux
Merge pull request italia#1084 from italia/loa-def
5991c59 
terms: better definition of LoA
@peppelinux
Merge pull request italia#1088 from italia/lts
bc34d25 
Authentic Source Onboarding Review
@giadas
Merge branch 'italia:versione-corrente' into iss-1076-wpmetadata
43af640
@giadas giadas added this to the 1.4.2 milestone Apr 17, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 17, 2026

📢 Documentation Preview 🔍

This PR comes from a forked repository, so the documentation preview was not automatically generated.

⚠️ For repository maintainers only:

You can generate a preview using the manual workflow, with PR Number: 1092 as an input.

Once generated, the preview link will be added as a comment to this PR.

@giadas giadas added the LTS label Apr 17, 2026
@giadas giadas requested a review from grausof April 17, 2026 12:59
manuraf
manuraf reviewed Apr 21, 2026
View reviewed changes
Comment thread docs/it/wallet-solution-metadata.rst Outdated
Comment thread docs/it/wallet-solution-metadata.rst Outdated
Comment thread docs/it/remote-flow.rst Outdated
@manuraf
Copy link
Copy Markdown
Collaborator

manuraf commented Apr 21, 2026

It might be worth revisiting this part for the test-plans-wallet-provider, for ex. WP_083a currently describes the wallet_metadata runtime including the authorization_endpoint.

With the changes introduced in this PR, the authorization_endpoint has been removed from the runtime wallet_metadata. As a result, few test cases seems to be no longer aligned with the updated specification.

@grausof grausof requested a review from Copilot April 24, 2026 09:05
Copilot AI reviewed Apr 24, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Wallet Provider Entity Configuration (EC) metadata structure and related documentation to align with issues #1075 and #1076 by removing the wallet_metadata wrapper and trimming presentation-time wallet metadata parameters.

Changes:

  • Updated the WP EC example to remove wallet_metadata and move wallet metadata fields to the wallet_solution top level.
  • Updated wallet solution metadata docs (IT/EN) to document the metadata fields at the top level (no wallet_metadata container).
  • Updated remote-flow docs (IT/EN) examples/tables to remove presentation-time wallet_metadata fields that are no longer required.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
examples/ec-wp.json Moves wallet metadata fields out of wallet_metadata into the wallet_solution object.
docs/it/wallet-solution-metadata.rst Updates IT documentation to reflect the new metadata structure and adds guidance about authorization endpoint links.
docs/it/remote-flow.rst Updates IT remote-flow examples/tables to reflect reduced wallet_metadata during presentation.
docs/en/wallet-solution-metadata.rst Updates EN documentation to reflect the new metadata structure and adds guidance about authorization endpoint links.
docs/en/remote-flow.rst Updates EN remote-flow examples/tables to reflect reduced wallet_metadata during presentation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/it/wallet-solution-metadata.rst Outdated
Comment thread docs/it/remote-flow.rst Outdated
Comment thread docs/it/wallet-solution-metadata.rst Outdated
Comment thread docs/it/wallet-solution-metadata.rst Outdated
Comment thread docs/it/wallet-solution-metadata.rst Outdated
Comment thread docs/en/wallet-solution-metadata.rst Outdated
Comment thread docs/en/wallet-solution-metadata.rst Outdated
@peppelinux peppelinux removed the LTS label Apr 28, 2026
@peppelinux peppelinux modified the milestones: 1.4.2, 1.5.0 Apr 28, 2026
@peppelinux peppelinux changed the base branch from versione-corrente to eudiw April 28, 2026 13:10
giadas
giadas commented Apr 28, 2026
View reviewed changes
Comment thread docs/it/wallet-solution-metadata.rst Outdated
@giadas @manuraf
Apply suggestions from code review
542f430 
Co-authored-by: Manuel Rafeli <manuel.rafeli@pagopa.it>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Giada Sciarretta <giada.sciarretta@gmail.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

📢 Documentation Preview 🔍

This PR comes from a forked repository, so the documentation preview was not automatically generated.

⚠️ For repository maintainers only:

You can generate a preview using the manual workflow, with PR Number: 1092 as an input.

Once generated, the preview link will be added as a comment to this PR.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 28, 2026

📢 Documentation Preview 🔍

This PR comes from a forked repository, so the documentation preview was not automatically generated.

⚠️ For repository maintainers only:

You can generate a preview using the manual workflow, with PR Number: 1092 as an input.

Once generated, the preview link will be added as a comment to this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@manuraf manuraf manuraf left review comments

Copilot code review Copilot Copilot left review comments

@grausof grausof Awaiting requested review from grausof

Assignees

No one assigned

Labels

breaking change eudiw

Projects

None yet

Milestone

1.5.0

Development

Successfully merging this pull request may close these issues.

6 participants

@giadas @manuraf @peppelinux @pcerqua-ipzs @m-basili