set default vals common fields for backend - Githubissues

ucsb-cs156-f23 / proj-happycows-f23-5pm-1

https://ucsb-cs156-f23.github.io/proj-happycows-f23-5pm-1/

0 stars 0 forks source link

set default vals common fields for backend #30

Closed haroldmo closed 10 months ago

haroldmo commented 10 months ago

Overview

In this PR, the commons variables are set from env variables (for default case)

Screenshots (Optional)

Generic placeholder image

Feedback Request (Optional)

Please give suggestions on the following:

This method might not be the best way to do things. What do you think?
The way this is implemented is the best I can think of. Is it acceptable?
Where should these things go? I put them here, but it feels out of place.

Future Possibilities (Optional)

Some possible points could be:

This is the beginning/part of (insert feature).
We can extend this to add (something beyond current functions).
This can make things easier to (insert project).

Validation (Optional)

Download this branch.
Run the project.
Go to this page.
Test this function.

Tests

[x] Backend Unit tests (mvn test) pass
[x] Backend Test coverage (mvn test jacoco:report) 100%
[x] Backend Mutation tests (mvn test pitest:mutationCoverage) 100%
[x] Frontend Unit tests (npm test) pass
[ ] Frontend Test coverage (npm run coverage) 100%
[ ] Frontend Mutation tests (npx stryker run) 100%
[ ] Frontend Linting (npx eslint --fix src)

Linked Issues

This is the first half of an issue for 10 points, so this won't close the issue