Skip to content
This repository was archived by the owner on Apr 6, 2026. It is now read-only.

NOISSUE - Add viewcert endpoint#14

Merged
dborovcanin merged 2 commits into
absmach:mainfrom
nyagamunene:add-view-endpoint
Sep 10, 2024
Merged

NOISSUE - Add viewcert endpoint#14
dborovcanin merged 2 commits into
absmach:mainfrom
nyagamunene:add-view-endpoint

Conversation

@nyagamunene

Copy link
Copy Markdown
Contributor

What type of PR is this?

This is a feature because it adds the following functionality: it add view cert endpoint.

What does this do?

It adds view cert endpoint instead of using download endpoint to view the end point

Which issue(s) does this PR fix/relate to?

N/A

Have you included tests for your changes?

Yes.

Did you document any new/modified features?

No. Will be done in different PR.

Notes

Signed-off-by: nyagamunene <stevenyaga2014@gmail.com>
@nyagamunene nyagamunene marked this pull request as ready for review September 10, 2024 17:11
Comment thread api/http/responses.go Outdated
}

type viewCertRes struct {
// Certificate certs.Certificate `json:"certificate"`

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Remove commented code.

Signed-off-by: nyagamunene <stevenyaga2014@gmail.com>
@dborovcanin dborovcanin merged commit d43d1c9 into absmach:main Sep 10, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants