Skip to content
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

ar - added instructorReport and updateCowHealth job placeholders #32

Merged
merged 1 commit into from Nov 27, 2022

Conversation

AlexanderRobles-CS
Copy link
Contributor

image

image

In this PR, I created the placeholders for "instructor report" and the "update cow health" jobs. In order to test this run the backend and frontend. Then log in using your credentials and finally checkout the jobs under the admins dropdown.

Copy link
Contributor

@cyril-wang cyril-wang left a comment

Choose a reason for hiding this comment

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

LGTM!

@cyril-wang cyril-wang added the f22-5pm-4 f22-5pm-4 label Nov 23, 2022
@pconrad pconrad added the STAFF: ready for review green on CI, has peer CR, no other obvious issues label Nov 25, 2022
@pconrad pconrad temporarily deployed to f22-5pm-4-happycows November 26, 2022 19:05 Inactive
Copy link
Contributor

@pconrad pconrad left a comment

Choose a reason for hiding this comment

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

LGTM

@pconrad pconrad merged commit 073e07d into main Nov 27, 2022
@pconrad pconrad added 5 5 pts and removed STAFF: ready for review green on CI, has peer CR, no other obvious issues labels Nov 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5 5 pts f22-5pm-4 f22-5pm-4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants