Skip to content

Conversation

@pierreTklein
Copy link
Member

Tickets:

List of changes:

Ran npm run docs, which updates the API documentation at docs.mchacks.ca/api

Type of change

Please delete options that are not relevant.

  • This change requires a documentation update

pierreTklein and others added 4 commits July 26, 2020 17:36
* fix: hasOwnProperty vulnerability (see https://eslint.org/docs/rules/no-prototype-builtins)
* feat: _id value to all route objects, starting at 100 and moving upwards. Add warning at top of file to maintain naming standard.
* feat: update role.constant.js to use the new _id attribute so that ids are deterministic between two runs of npm run seed.
* feat: update architecture documentation
* feat: Add HACKER_UPDATE_BATCH success message

* feat: Add promise.allsettled so that we can wait for many promises to complete

* feat: Add findByHackerId function to Account

* feat: Add updatedHackerBatch controller function, generalize validation function, fix router

* fix: resolve bug where requests hang when response is null

* feat: set route apiVersion number to be 3.0.0

* feat: create parseAcceptBatch

Co-authored-by: Pierre Theo Klein <pierre.klein@mail.mcgill.ca>
@pierreTklein pierreTklein changed the base branch from dev to develop July 31, 2020 04:41
@loreina loreina changed the base branch from develop to dev July 31, 2020 04:47
@loreina loreina mentioned this pull request Jul 31, 2020
1 task
@loreina loreina changed the title Feature/update docs docs: update docs for v3 Jul 31, 2020
@loreina loreina changed the title docs: update docs for v3 docs: update docs for v3.0.0 Jul 31, 2020
@loreina loreina merged commit bd04b3b into dev Jul 31, 2020
@loreina loreina deleted the feature/update-docs branch July 31, 2020 19:14
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.

4 participants