Skip to content

PIN-6617 - M2M Api DELETE /tenants/:tenantId/certifiedAttributes #1821

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 17 commits into
base: pin-6616_POST-assign-certified-attribute
Choose a base branch
from
Open

PIN-6617 - M2M Api DELETE /tenants/:tenantId/certifiedAttributes #1821

wants to merge 17 commits into from

Conversation

Carminepo2
Copy link
Contributor

@Carminepo2 Carminepo2 commented Apr 22, 2025 •
edited
Loading

← back

Merge after #1819
Closes PIN-6617

Description

Added DELETE /tenants/:tenantId/certifiedAttributes route and service.
Also added the return of Tenant from the revokeCertifiedAttributeById tenant process service.

Double-check errors

  • Check errors returned by the process API calls (messages should be meaningful, clean, in correct English, etc.)
  • Check spec errors: it should list all non-500 errors, also the ones that pass through from the process

Testing

Checklist:

  • Bruno call added to collection
  • Smoke test Bruno (attach result screenshot)
  • Supertest /api tests
  • /integration tests for the logic in the service
Screenshot 2025-04-24 alle 13 32 31

@Carminepo2 Carminepo2 mentioned this pull request Apr 22, 2025
Draft
5 tasks
Carminepo2 added 10 commits April 22, 2025 16:41
@Carminepo2
Merge branch 'pin-6616_POST-assign-certified-attribute' into pin-6617…
6abc632 
…_DELETE-revoke-certified-attribute
@Carminepo2
Merge branch 'pin-6616_POST-assign-certified-attribute' into pin-6617…
f79c0b5 
…_DELETE-revoke-certified-attribute
@Carminepo2
Merge branch 'pin-6616_POST-assign-certified-attribute' into pin-6617…
009d94b 
…_DELETE-revoke-certified-attribute
@Carminepo2
Update authorization
d4da7bd
@Carminepo2
Merge branch 'pin-6616_POST-assign-certified-attribute' into pin-6617…
137834e 
…_DELETE-revoke-certified-attribute
@Carminepo2
Merge branch 'pin-6616_POST-assign-certified-attribute' into pin-6617…
1b0ae32 
…_DELETE-revoke-certified-attribute
@Carminepo2
Added metadata to revoke certified attribute tenant process route
86aa88e
@Carminepo2
Merge branch 'pin-6617_DELETE-revoke-certified-attribute' of https://…
7a91fe4 
…github.com/pagopa/interop-be-monorepo into pin-6617_DELETE-revoke-certified-attribute
@Carminepo2
Update Revoke Certified Attribute.bru
bc90e0b
@Carminepo2
Merge branch 'pin-6616_POST-assign-certified-attribute' into pin-6617…
c8bece7 
…_DELETE-revoke-certified-attribute
@Carminepo2 Carminepo2 marked this pull request as ready for review April 30, 2025 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ecamellini ecamellini Awaiting requested review from ecamellini

@zerbiniandrea zerbiniandrea Awaiting requested review from zerbiniandrea

@beetlecrunch beetlecrunch Awaiting requested review from beetlecrunch beetlecrunch is a code owner

@galales galales Awaiting requested review from galales galales is a code owner

@AsterITA AsterITA Awaiting requested review from AsterITA AsterITA is a code owner

@taglioni-r taglioni-r Awaiting requested review from taglioni-r taglioni-r is a code owner

Assignees
No one assigned
Labels
api-v2 tenant-process
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Carminepo2