Phonebooks - Refactor API routes to respect multi-tenant and UUID

Description

  • Remove tenant from URL

    • Replace /tenants/{tenant_name}/phonebooks/{id} for /phonebook/{uuid}

    • Must pass tenant inside headers

  • Remove usage of ID in favor of UUID

    • Phonebook should be UUID

    • Contact have ID field but it’s an UUID (rename)

Questions

  • How we deprecate a route

Requirements

  • keep previous API fully functional

  • add uuid identifier for phonebook

  • add new phonebook API with phonebook uuid as identifier

  • support multitenancy transparently through HTTP headers, token

    • support “hierarchical” tenant access(i.e. with ?recursive=true query parameter allowing to access resources from all subtenant tree)

Zendesk Ticket IDs

None
100% Done
Loading...

Activity

Show:

Charles Langlois July 31, 2023 at 1:15 PM

PR merged.

Charles Langlois July 20, 2023 at 3:19 PM

First pass of review processed. A second pass to approve changes and add feedback would be valuable.

Charles Langlois July 7, 2023 at 2:14 PM

ACL changes have no direct consequences.

Charles Langlois July 6, 2023 at 1:21 PM

  • Add warning logger on deprecated routes

Charles Langlois July 4, 2023 at 1:33 PM

  • Must deal with new acls?

Done

Details

Priority

Assignee

Reporter

Approvers

François Blackburn

Fix versions

Sprint

Labels

Zendesk Support

Created May 18, 2023 at 3:30 PM
Updated August 9, 2023 at 3:15 PM
Resolved July 31, 2023 at 1:15 PM