-
Notifications
You must be signed in to change notification settings - Fork 14
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
#4281 - SDPR API - Student search result - Student details #4376
base: main
Are you sure you want to change the base?
Conversation
|
@@ -0,0 +1,99 @@ | |||
import { AddressAPIOutDTO } from "../../models/common.dto"; |
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.
Please import it from the index.
import { AddressAPIOutDTO } from "../..";
// If the student is found in SIMS, return the student details. | ||
// Otherwise, return the legacy student details. | ||
const studentDetails = student | ||
? this.transformStudentDetails(student) | ||
: this.transformLegacyStudentDetails(sfasIndividual); |
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.
Is this a temporary solution or is it already the final approach to return student data?
I was assuming that the entire dataset of student + application data (a single query) would be executed (in parallel), targeting SIMS data and SFAS data. From the complete result of these two queries a "merge" would happen.
Is the idea that sfasIndividualPromise
and studentPromise
will be expanded?
* @param student student. | ||
* @returns student details. | ||
*/ | ||
private transformStudentDetails(student: Student): StudentDetails { |
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.
For my understanding, will these transform
methods be moved into some service to keep the controller "clean" in some upcoming PR?
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.
Great work and thanks for enabling the secret auth for E2Es. Please take a look at the comments, mostly questions for clarification.
PR Scope
Student search result - Student details
StudentInformationService
to get all student related information details based on both SIMS and SFAS.E2E Test Helpers
client id
andclient secret
to obtain the token.Repository secret
that will be used by E2E tests forexternal
controller.E2E Tests
Upcoming PR