Skip to content

Commit

Permalink
Added ADR for Identity to move to TRS (#934)
Browse files Browse the repository at this point in the history
  • Loading branch information
aje54 authored Dec 8, 2023
1 parent 74c4e3c commit 804869d
Showing 1 changed file with 31 additions and 0 deletions.
31 changes: 31 additions & 0 deletions docs/adr/0001-move-id-into-trs.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,31 @@
# 1. Record architecture decisions

Date: 2023-11-30

## Status

Accepted

## Context

With the adoption of Gov.OneLogin into our services, it renders a proportion of Teaching Identity's functionality either duplicated or obsolete. Therefore, we reviewed how best to enable our services to
integrate with Gov.OneLogin and also provide the necessary teaching record authorisation (that Teaching Identity currently provides). We chose to keep the OAUTH based journey that allows services to pass a set of PII data to be checked against the database of qualified teachers (soon to be Teaching Record System).

## Decision

* Keep OAUTH journey code to authorise access to teaching record
* Move the code from Get an Identity into Teaching record system
* Archive / stop updating the Get an Identity code base
* Remove the uid (of teaching identity) from use within teaching services
* Use the GOV.OneLogin uid to join identities across services


## Consequences
Register for a national qualification and claim teacher payments services will need to make relatively mionor changes to their matching code.

We will need to re-factor Teaching Record Sytem accordingly to provide teaching record authorisation via the OAUTH journey (copied from Get an Identity)

Repos Affected:
https://github.com/DFE-Digital/get-an-identity
https://github.com/DFE-Digital/teaching-record-system

0 comments on commit 804869d

Please sign in to comment.