Skip to content
This repository has been archived by the owner on Oct 6, 2023. It is now read-only.

#121: Implement the UI to connect manage the certificate of the PKI #127

Merged
merged 6 commits into from
Jul 5, 2022

Conversation

tuanSAG
Copy link
Contributor

@tuanSAG tuanSAG commented Jun 7, 2022

added tab Managed Certificates and Button

@tuanSAG tuanSAG added this to the Priority 0 milestone Jun 7, 2022
@tuanSAG tuanSAG self-assigned this Jun 7, 2022
@tuanSAG tuanSAG added the FE Frontend related implementation label Jun 7, 2022
@dirk-peter-c8y dirk-peter-c8y removed their request for review June 7, 2022 11:40
Copy link

@dirk-peter-c8y dirk-peter-c8y left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

overall only small changes needed.

however:

  • i'm missing error-handling for requests. don't just expect requests to work as expetced
  • i could not find any user feedback implemented (eg. loading indicators, toast-messages)

…Adjusted after Code Review/ Added Button to Create Keypair/ Added Service with DevicedId
@dirk-peter-c8y dirk-peter-c8y marked this pull request as draft June 21, 2022 09:19
pkiintergration-fe/app.module.ts Outdated Show resolved Hide resolved
pkiintergration-fe/app.module.ts Outdated Show resolved Hide resolved
pkiintergration-fe/src/hooks/action.ts Outdated Show resolved Hide resolved
pkiintergration-fe/src/hooks/action.ts Outdated Show resolved Hide resolved
pkiintergration-fe/src/hooks/hooks.module.ts Outdated Show resolved Hide resolved
…Added Admin Applikation/ Code adjusted after Code Review/ Fixed error in MicroService
@@ -1 +1 @@
<button id="action-btn" class="btn-link" type="button" (click)="requestPubKey()">Create Keypair</button>
<button id="action-btn" class="btn-link" type="button" style="text-align: left;" (click)="requestPubKey()">Set device connection method</button>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo: do not use inline style

Comment on lines 12 to 24
// device: IManagedObject;
alertService: AlertService;
ActivatedRoute: ActivatedRoute;

constructor(private keypairService: CreateKeyPairService, private route: ActivatedRoute) {}
constructor(private createKeypairService: CreateKeyPairService, private activatedRoute: ActivatedRoute) {}

ngOnInit() {
const snapshot = this.route.snapshot;
const snapshot = this.activatedRoute.snapshot;
if (snapshot && snapshot.firstChild && snapshot.firstChild.data) {
this.deviceId = snapshot.firstChild.data.contextData.id;
}

console.log('devideid:', this.deviceId);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo: mark page as "work in progress" or resolve previously mentioned issues

Comment on lines 29 to 30
<button class="btn btn-default" translate="" (click)="bsModalRef.hide()">Cancel</button>
<button type="submit" class="btn btn-primary" (click)="bsModalRef.hide()">Generate</button>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

question: why do both buttons trigger the same action/function - where is the difference?

@kreinhar kreinhar marked this pull request as ready for review July 5, 2022 10:39
@kreinhar kreinhar merged commit 4d22864 into main Jul 5, 2022
@kreinhar kreinhar deleted the pki-intergration-frontend branch July 5, 2022 10:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FE Frontend related implementation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants