Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
dlloyd1-ch created a review comment on a pull request on companieshouse/limited-partnerships-api
The summary for the healthcheck still needs updating. It's not creating a registration submission. Suggest something like: `Health check URL returns 200 if service is running`
dlloyd1-ch created a review on a pull request on companieshouse/limited-partnerships-api
mwestacott created a comment on a pull request on companieshouse/limited-partnerships-api
LGTM
mattch1 created a review comment on a pull request on companieshouse/limited-partnerships-api
Updated to say "get"
mattch1 created a review on a pull request on companieshouse/limited-partnerships-api
mattch1 pushed 1 commit to lp-119-intial-api-spec companieshouse/limited-partnerships-api
- hc endpoint should be a get 2726241
dlloyd1-ch created a review comment on a pull request on companieshouse/limited-partnerships-api
Needs changing to "get". Also the summary needs updating.
dlloyd1-ch created a review on a pull request on companieshouse/limited-partnerships-api
mattch1 created a review comment on a pull request on companieshouse/limited-partnerships-api
Cut this back.
mattch1 created a review on a pull request on companieshouse/limited-partnerships-api
mattch1 pushed 1 commit to lp-119-intial-api-spec companieshouse/limited-partnerships-api
- Cut back spec to show only the healthcheck endpoint c983c95
mwestacott created a review comment on a pull request on companieshouse/limited-partnerships-api
I think only the healthcheck end-point should feature in the spec for the moment. We add the others when we come to implement the end-points in controllers.
mwestacott created a review on a pull request on companieshouse/limited-partnerships-api
mattch1 created a review comment on a pull request on companieshouse/limited-partnerships-api
Updated spec endpoint using a hyphen for limited-partnership.
mattch1 created a review on a pull request on companieshouse/limited-partnerships-api
mattch1 created a review comment on a pull request on companieshouse/limited-partnerships-api
Updated to use the actuator by adding application.properties config.
mattch1 created a review on a pull request on companieshouse/limited-partnerships-api
mattch1 pushed 1 commit to lp-119-intial-api-spec companieshouse/limited-partnerships-api
- Use actuator for the healthcheck endpoint 808e8ad
mattch1 created a comment on a pull request on companieshouse/limited-partnerships-api
Updated the schema to reflect https://companieshouse.atlassian.net/wiki/spaces/Arch/pages/4722491429/LP+Service+Design As we do not have schema details yet that will have to be added later.
mattch1 pushed 1 commit to lp-119-intial-api-spec companieshouse/limited-partnerships-api
- LP-119 Update schema to reflect design document. 86cb1b4