The main point of this commit is to change the default ABE_URL value to https://abe-dev.olin.build, so that newly-minted PR apps point to dev instead of prod.
Also took the opportunity to add repository and success_url properties as documented here, and remove some of the optional empty values.
Required
Changes must conform to these requirements:
[x] yarn test passes. All new and existing tests pass.
[x] yarn lint passes. All new code follows the code style of this project.
Aspirational
We don't yet require these, but they are nice to have:
[n/a] New code is covered by new or existing tests.
[n/a] Changed code is covered by new or existing tests.
The main point of this commit is to change the default
ABE_URLvalue tohttps://abe-dev.olin.build, so that newly-minted PR apps point to dev instead of prod.Also took the opportunity to add
repositoryandsuccess_urlproperties as documented here, and remove some of the optional empty values.Required
Changes must conform to these requirements:
yarn testpasses. All new and existing tests pass.yarn lintpasses. All new code follows the code style of this project.Aspirational
We don't yet require these, but they are nice to have: