Skip to content

data: add Aruba (nr 219) - #11

Merged
thodoris merged 1 commit into
mainfrom
wallet-update-219-20260923
Sep 23, 2026
Merged

thodoris merged 1 commit into
mainfrom
wallet-update-219-20260923

Conversation

@marco-pancioni-aruba

Copy link
Copy Markdown
Contributor

data: add Aruba (nr 219)

(cherry picked from commit 0c4adb7ff68e16cc680c04af4ffa9d2b1c133706)

@thodoris thodoris left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Validated locally: CSV valid (39 -> 40 entries), 15 columns, ID 219 unique, vocabulary values OK, build passes.

@thodoris
thodoris merged commit e0ba481 into main Sep 23, 2026
1 check passed
@thodoris
thodoris deleted the wallet-update-219-20260923 branch September 23, 2026 09:31
@thodoris

Copy link
Copy Markdown
Contributor

Hi @marco-pancioni-aruba,

Thank you for adding Aruba to the inventory. One correction is needed in wallet-capabilities/public/wallet capabilities.csv: the "nr in Portal" should be 227, which is the portal number of ARUBA SPA in the Grant Agreement. Number 219 belongs to another partner. Can you check ?

Could you please also check the short name? The entry says "Aruba PEC Spa", while the partner in the GA is Aruba S.p.A. If the wallet is provided by Aruba S.p.A., please use "Aruba".

As this PR is already merged, please open a new PR with these changes and I will review it.

Kind regards,
Thodoris

@marco-pancioni-aruba

Copy link
Copy Markdown
Contributor Author

Hi @thodoris, I've updated Aruba's data as requested. Check #12 if my update is right.

Thanks in advance,

Marco

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants