Skip to content
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.

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

Feature/36 variable recurring payments for 3 1 8 #469

Merged
merged 30 commits into from
Dec 15, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
7caee3d
45: Add VRP generated API and Controllers
BohoCode Nov 18, 2021
bb7231f
commit before rebase master
jsanhc Nov 26, 2021
f5dddd8
45: Use latest uk-datamodel with 3.1.8r5 swagger models
BohoCode Nov 26, 2021
7026060
Release candidate: prepare release 1.5.6
BohoCode Nov 26, 2021
b725a10
Release candidate: prepare for next development iteration
BohoCode Nov 26, 2021
6d1e9e3
after rebase master
jsanhc Nov 26, 2021
b66ad4c
47: vrp consent controllers
jsanhc Nov 26, 2021
3d7bd17
47: VRP consent controllers
jsanhc Nov 26, 2021
6401e76
Intermediate checkin with new 3.1.9r3 datamodels (#462)
BohoCode Nov 30, 2021
e76eca6
47: vrp consent controllers (#463)
jsanhc Dec 1, 2021
a3e4aed
47: vrp payment consent controller (#464)
jsanhc Dec 3, 2021
86b7e5c
48: Develop DomesticVrpApi and controllers
BohoCode Nov 30, 2021
8b4fca9
Merge pull request #465 from OpenBankingToolkit/issue/48-implementati…
BohoCode Dec 3, 2021
cc09846
Issue/47 implementation of vrp payment consent controller (#466)
jsanhc Dec 3, 2021
93b5073
36-47: VRP payment consent controller
jsanhc Dec 6, 2021
8e45388
36-47: VRP consent controller flow
jsanhc Dec 8, 2021
b46ab98
VRP Funds confirmation
jsanhc Dec 8, 2021
b69e90a
Fix Domestic consent controller constructor to delete unused service
jsanhc Dec 8, 2021
1d83155
Fix integration tests for VRP funds confirmation
jsanhc Dec 8, 2021
ca0b17e
36: Add filters to test vrp Payments risk and initiation
BohoCode Dec 9, 2021
742e63b
Merge pull request #467 from OpenBankingToolkit/feature/36-variable-r…
BohoCode Dec 9, 2021
16e0e47
36: Further VRP request filters
BohoCode Dec 9, 2021
f0f1814
Merge pull request #468 from OpenBankingToolkit/feature/36-variable-r…
BohoCode Dec 9, 2021
80cdf70
36-48: VRP payment controller validations
jsanhc Dec 13, 2021
a0a2e60
Vrp payment submission controller and integration tests
jsanhc Dec 13, 2021
2286430
vrp payment contrller gateway server implementations
jsanhc Dec 14, 2021
2672d9d
Fix node version github workflow to build and tests the UIs
jsanhc Dec 14, 2021
bdbc96f
Update version to 1.6.0
jsanhc Dec 15, 2021
5cf6b45
Merge branch 'master' into feature/36-variable-recurring-payments-for…
jsanhc Dec 15, 2021
150313d
Fix npm tests for VRP payment component
jsanhc Dec 15, 2021
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions .github/workflows/ui-release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,10 @@ jobs:
steps:
- name: Checkout Repo
uses: actions/checkout@v2
- name: setup node version 10
uses: actions/setup-node@v2
with:
node-version: '10'
- name: Run tests
working-directory: ./forgerock-openbanking-ui
run: |
Expand Down
4 changes: 4 additions & 0 deletions .github/workflows/ui.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@ jobs:
steps:
- name: Checkout Repo
uses: actions/checkout@v2
- name: setup node version 10
uses: actions/setup-node@v2
with:
node-version: '10'
- name: Run tests
working-directory: ./forgerock-openbanking-ui
run: |
Expand Down
1,167 changes: 590 additions & 577 deletions CHANGELOG.md

Large diffs are not rendered by default.

8 changes: 7 additions & 1 deletion forgerock-openbanking-ui/README.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,13 @@
## Run the app
- bank (default angular app)
- manual-onboarding
### Run local app
### Run local bank app
```shell
# bank app
npm run build.bank.themes
npm run serve.bank.local
```
### Run other scripts (@see package.json)
- `npm run build.{app/project}.themes`
- `npm run serve.{app/project}`
```shell
Expand Down
135 changes: 115 additions & 20 deletions forgerock-openbanking-ui/angular.json
Original file line number Diff line number Diff line change
@@ -1,15 +1,18 @@
{
"$schema": "./node_modules/@angular/cli/lib/config/schema.json",
"version": 1,
"defaultProject": "bank",
"newProjectRoot": "projects",
"projects": {
"bank": {
"root": "projects/bank/",
"sourceRoot": "projects/bank/src",
"projectType": "application",
"prefix": "app",
"schematics": { "@schematics/angular:component": { "styleext": "scss" } },
"schematics": {
"@schematics/angular:component": {
"styleext": "scss"
}
},
"architect": {
"build": {
"builder": "ngx-build-plus:browser",
Expand All @@ -20,10 +23,23 @@
"main": "projects/bank/src/main.ts",
"polyfills": "projects/bank/src/polyfills.ts",
"tsConfig": "projects/bank/tsconfig.app.json",
"assets": ["projects/bank/src/favicon.ico", "projects/bank/src/assets"],
"styles": ["node_modules/cookieconsent/build/cookieconsent.min.css", "projects/bank/src/styles.scss"],
"scripts": ["node_modules/cookieconsent/build/cookieconsent.min.js"],
"stylePreprocessorOptions": { "includePaths": ["utils/scss", "projects/bank/src/scss"] },
"assets": [
"projects/bank/src/favicon.ico",
"projects/bank/src/assets"
],
"styles": [
"node_modules/cookieconsent/build/cookieconsent.min.css",
"projects/bank/src/styles.scss"
],
"scripts": [
"node_modules/cookieconsent/build/cookieconsent.min.js"
],
"stylePreprocessorOptions": {
"includePaths": [
"utils/scss",
"projects/bank/src/scss"
]
},
"es5BrowserSupport": true
},
"configurations": {
Expand All @@ -46,8 +62,26 @@
"statsJson": true,
"es5BrowserSupport": true
},
"local": {
"fileReplacements": [
{
"replace": "projects/bank/src/environments/environment.ts",
"with": "projects/bank/src/environments/environment.local.ts"
}
]
},
"wiremock": {
"fileReplacements": [
{
"replace": "projects/bank/src/environments/environment.ts",
"with": "projects/bank/src/environments/environment.wiremock.ts"
}
]
},
"ui-integ-native": {
"options": { "main": "projects/bank/src/main-native.ts" },
"options": {
"main": "projects/bank/src/main-native.ts"
},
"fileReplacements": [
{
"replace": "projects/bank/src/environments/environment.ts",
Expand Down Expand Up @@ -75,18 +109,46 @@
"sslCert": "ssl/all.dev-ob.forgerock.financial.crt",
"sslKey": "ssl/k-all.dev-ob.forgerock.financial.key"
},
"configurations": { "production": { "browserTarget": "bank:build:production" } }
"configurations": {
"production": {
"browserTarget": "bank:build:production"
},
"local": {
"browserTarget": "bank:build:local"
},
"wiremock": {
"browserTarget": "bank:build:wiremock"
}
}
},
"extract-i18n": {
"builder": "@angular-devkit/build-angular:extract-i18n",
"options": { "browserTarget": "bank:build" }
"options": {
"browserTarget": "bank:build"
}
},
"test": {
"builder": "@angular-builders/jest:run",
"options": {
"no-cache": true,
"reporters": [
"default"
]
}
},
"test": { "builder": "@angular-builders/jest:run", "options": { "no-cache": true, "reporters": ["default"] } },
"lint": {
"builder": "@angular-devkit/build-angular:tslint",
"options": {
"tsConfig": ["projects/bank/tsconfig.app.json", "projects/bank/tsconfig.spec.json"],
"exclude": ["**/node_modules/**", "projects/@fuse/**", "**/*.spec.ts", "**/*.storybook.ts"]
"tsConfig": [
"projects/bank/tsconfig.app.json",
"projects/bank/tsconfig.spec.json"
],
"exclude": [
"**/node_modules/**",
"projects/@fuse/**",
"**/*.spec.ts",
"**/*.storybook.ts"
]
}
}
}
Expand All @@ -96,7 +158,11 @@
"sourceRoot": "projects/manual-onboarding/src",
"projectType": "application",
"prefix": "app",
"schematics": { "@schematics/angular:component": { "styleext": "scss" } },
"schematics": {
"@schematics/angular:component": {
"styleext": "scss"
}
},
"architect": {
"build": {
"builder": "ngx-build-plus:browser",
Expand All @@ -107,13 +173,23 @@
"main": "projects/manual-onboarding/src/main.ts",
"polyfills": "projects/manual-onboarding/src/polyfills.ts",
"tsConfig": "projects/manual-onboarding/tsconfig.app.json",
"assets": ["projects/manual-onboarding/src/favicon.ico", "projects/manual-onboarding/src/assets"],
"assets": [
"projects/manual-onboarding/src/favicon.ico",
"projects/manual-onboarding/src/assets"
],
"styles": [
"node_modules/cookieconsent/build/cookieconsent.min.css",
"projects/manual-onboarding/src/styles.scss"
],
"scripts": ["node_modules/cookieconsent/build/cookieconsent.min.js"],
"stylePreprocessorOptions": { "includePaths": ["utils/scss", "projects/manual-onboarding/src/scss"] },
"scripts": [
"node_modules/cookieconsent/build/cookieconsent.min.js"
],
"stylePreprocessorOptions": {
"includePaths": [
"utils/scss",
"projects/manual-onboarding/src/scss"
]
},
"es5BrowserSupport": true
},
"configurations": {
Expand Down Expand Up @@ -156,21 +232,40 @@
"sslCert": "ssl/all.dev-ob.forgerock.financial.crt",
"sslKey": "ssl/k-all.dev-ob.forgerock.financial.key"
},
"configurations": { "production": { "browserTarget": "manual-onboarding:build:production" } }
"configurations": {
"production": {
"browserTarget": "manual-onboarding:build:production"
}
}
},
"extract-i18n": {
"builder": "@angular-devkit/build-angular:extract-i18n",
"options": { "browserTarget": "manual-onboarding:build" }
"options": {
"browserTarget": "manual-onboarding:build"
}
},
"test": {
"builder": "@angular-builders/jest:run",
"options": {
"no-cache": true,
"reporters": [
"default"
]
}
},
"test": { "builder": "@angular-builders/jest:run", "options": { "no-cache": true, "reporters": ["default"] } },
"lint": {
"builder": "@angular-devkit/build-angular:tslint",
"options": {
"tsConfig": [
"projects/manual-onboarding/tsconfig.app.json",
"projects/manual-onboarding/tsconfig.spec.json"
],
"exclude": ["**/node_modules/**", "projects/@fuse/**", "**/*.spec.ts", "**/*.storybook.ts"]
"exclude": [
"**/node_modules/**",
"projects/@fuse/**",
"**/*.spec.ts",
"**/*.storybook.ts"
]
}
}
}
Expand Down
2 changes: 2 additions & 0 deletions forgerock-openbanking-ui/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@
"report.bank": "npm run build.bank.forgerock && webpack-bundle-analyzer dist/forgerock/stats.json",
"build.bank.forgerock": "ng build --project=bank --prod --configuration=forgerock --output-path dist/forgerock --statsJson --extra-webpack-config webpack.extra.js",
"build.bank.themes": "openbanking-ui-cli buildThemes --project bank",
"build.bank.wiremock": "ng serve bank -c wiremock",
"build.bank.local": "ng serve bank -c local",
"=================REGISTER APP=================": "",
"bundleSize.manual-onboarding": "openbanking-ui-cli appSize --theme forgerock --project manual-onboarding",
"serve.manual-onboarding": "ng serve manual-onboarding",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ import { SubmitBoxComponentModule } from '../components/submit-box/submit-box.mo
import { AccountCheckboxModule } from '../components/account-checkbox/account-checkbox.module';

import { AccountComponent } from './account.component';
import {ConsentDecision} from "bank/src/app/types/ConsentDecision";

describe('app:bank AccountComponent', () => {
let component: AccountComponent;
Expand Down Expand Up @@ -55,7 +56,7 @@ describe('app:bank AccountComponent', () => {
fixture.detectChanges();

expect(component.formSubmit.emit).toHaveBeenCalledWith({
decision: 'deny',
decision: ConsentDecision.DENY,
sharedAccounts: []
});
});
Expand All @@ -67,7 +68,7 @@ describe('app:bank AccountComponent', () => {
fixture.detectChanges();

expect(component.formSubmit.emit).toHaveBeenCalledWith({
decision: 'deny',
decision: ConsentDecision.DENY,
sharedAccounts: []
});
});
Expand All @@ -79,7 +80,7 @@ describe('app:bank AccountComponent', () => {
fixture.detectChanges();

expect(component.formSubmit.emit).toHaveBeenCalledWith({
decision: 'allow',
decision: ConsentDecision.ALLOW,
sharedAccounts: []
});
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ export class AccountComponent implements OnInit {
type: ItemType.STRING,
payload: {
label: 'CONSENT.ACCOUNT.APPLICATION',
value: this.response.aispName,
value: this.response.pispName,
cssClass: 'account-merchantName'
}
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
<div>
<div>{{ account.account?.Nickname }}</div>
<div class="secondary-text account-number" *ngIf="account.account?.Account">
{{ (account.account?.Account)[0] | AccountNumberFormat }}
{{ (account.account?.Account)[0] | AccountFormat }}
</div>
</div>

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
<div>
<div>{{ account.account?.Nickname }}</div>
<div class="secondary-text account-number" *ngIf="account.account?.Account">
{{ (account.account?.Account)[0] | AccountNumberFormat }}
{{ (account.account?.Account)[0] | AccountFormat }}
</div>
<div *ngIf="account.balances">
<div class="balance" *ngFor="let balance of account.balances">{{ balance | BalanceFormat }}</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ export class CancelComponent implements OnInit {
type: ItemType.STRING,
payload: {
label: 'COMPONENT.CANCEL.APPLICATION',
value: this.response.aispName,
value: this.response.pispName,
cssClass: 'cof-merchantName'
}
}
Expand Down
Loading