Skip to content

Conversation

@mxmeinhold
Copy link
Collaborator

@mxmeinhold mxmeinhold commented Oct 30, 2019

New routes to sync freshman and to create packets in order to facilitate apply.csh

Closes #178

Copy link
Member

@JoelEager JoelEager left a 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. Though in the future we may want to support batch creation of packets since that would be the "normal" use case.

@mxmeinhold
Copy link
Collaborator Author

I think handling the payload as a list is best, so I'd like to get that in before this get merged.

Copy link
Member

@devinmatte devinmatte left a comment

Choose a reason for hiding this comment

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

Sorry for the delay

@devinmatte devinmatte merged commit 731fb6f into ComputerScienceHouse:develop Nov 26, 2019
@mxmeinhold mxmeinhold deleted the apply-routes branch November 26, 2019 19:16
@mxmeinhold mxmeinhold mentioned this pull request Mar 7, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

API Endpoint to generate a packet

3 participants