...
| Info |
|---|
Add your comments directly to the page. Include links to any relevant research, data, or feedback. |
| Page Properties |
|---|
|
| Status | | Status |
|---|
| colour | Yellow |
|---|
| title | In progress |
|---|
|
|
|---|
| Impact | |
|---|
| Driver | |
|---|
| Approver | |
|---|
| Contributors | |
|---|
| Informed | TIS Team |
|---|
| Due date | |
|---|
| Outcome | |
|---|
|
Background
There's a lot of tech debt across TIS. In FE this is the case too.
...
- Different versions of Angular in use
- Removal of dependency on jHipster scaffolding
- Aligning the filing structure to match best practice for Angular 5
- Tidy up the SCSS folder and code
- Where to apply global and local styles
- Tidying up of a plethora of tests and test approaches
- NGRX - observable
Options considered
| Option 1: | Option 2: | Option 3: | Option 4: |
|---|
| Bringing all elements in TIS up to the same, current version of Angular | V5 | V6 | V7 | V8 |
|---|
| Pros and cons |  Admins UI is v5 and the biggest element of TIS. Getting everything to v5 would mean we could then upgrade all of TIS to later versions of Angular more easily
 v5 is already 2 full versions behind the current stable version of Angular (v7). Admins UI contains lots of non-Angular v5 code within it that will need to be converted to Angular v5 first
| More modern version of Angular opens up new features and addresses issues from v5 Admins UI is now Angular v6, but bringing Reval up to v6 is a large task | More modern version of Angular opens up new features and addresses issues from v6 No element is using Angular |
|---|
v6 v7 yet, so bringing all of TIS up to |
v6 v7 is a huge task | More modern version of Angular opens up new features and addresses issues from |
v6v7 No element is using Angular |
v7 v8 yet, so bringing all of TIS up to |
v7 v8 is a huge task |
| Estimated cost | | | | |
|---|
| Status | | Status |
|---|
| |
|---|
| colour | Yellow |
|---|
| title | In progress |
|---|
|
- Admins UI , Reval | | Status |
|---|
| |
|---|
| colour | YELLOW |
|---|
| title | in progress |
|---|
|
- Admins UI , Reval | | |
|---|
Action items
- Update Admins UI code to Angular v5
- Update Admins UI code to Angular v6
- Update Admins UI code to Angular v7
- Update Admins UI code to Angular v8
- TISNEW-2498: Re-write Angular 1.2/1.5 apps (Reval | Concerns Log | Notifications | Common Components) to Angular v5, and upgrade to v6, v7 and v8
- TISNEW-2503: Remove dependencies on jHipster scaffolding
- TISNEW-1650: Move to standardised file structure for Angular v5, v6, v7, v8 - (Panos Paralakis (Unlicensed), Oladimeji Onalaja (Unlicensed), Tuomas Alahäivälä (Unlicensed) - does this need to be defined clearly somewhere, in order to give you all a consistent reference point?)
- Tidy up SCSS folder, files and code (Panos Paralakis (Unlicensed), Oladimeji Onalaja (Unlicensed), Tuomas Alahäivälä (Unlicensed) - does this need to be defined clearly somewhere, in order to give you all a consistent reference point?)
- Decide when global components are preferable, and where TIS would benefit from local styles
- Tidy up tests - protractor, cucumber, etc (Panos Paralakis (Unlicensed), Oladimeji Onalaja (Unlicensed), Tuomas Alahäivälä (Unlicensed) - does this need to be defined clearly somewhere, in order to give you all a consistent reference point?)
- NGRX - observable (Panos Paralakis (Unlicensed), Oladimeji Onalaja (Unlicensed), Tuomas Alahäivälä (Unlicensed) - please urn turn this item into a specific action)
...