-
Notifications
You must be signed in to change notification settings - Fork 10
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
Create controller API for fellowship #26
Conversation
backend/src/main/java/org/sefglobal/core/exception/CertificateNotFoundException.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/model/Certificate.java
Outdated
Show resolved
Hide resolved
db2d02b
to
dbf24f3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me! 😊
Add JavaDoc comments for the service layer :)
backend/src/main/java/org/sefglobal/core/controller/CertificateController.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/service/CertificateService.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/controller/CertificateController.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Clean code! 🔥
@YohanAvishke Could you please review this? |
backend/src/main/java/org/sefglobal/core/controller/CertificateController.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/controller/CertificateController.java
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/model/Certificate.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/service/CertificateService.java
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/service/CertificateService.java
Show resolved
Hide resolved
9c72edf
to
c60b3ec
Compare
backend/src/main/java/org/sefglobal/core/controller/CertificateController.java
Outdated
Show resolved
Hide resolved
backend/src/main/java/org/sefglobal/core/model/Certificate.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Perfect!
Purpose
The purpose of this PR is to fix #25
Goals
The goal is to create an API to be functioned as the sef-fellowship certificate generator.
Approach
Created all the necessary files and implemented the API endpoints adhering to the above requirements in the issue.
Checklist
Test environment
Learning