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

Adds registration management to the API. #23

Merged
merged 14 commits into from
Mar 26, 2019

Conversation

Tazaf
Copy link
Contributor

@Tazaf Tazaf commented Sep 13, 2018

Registration being the process of someone registering with their
email to receive updates, notifications or news on the BioPocket project.

Adds a new table registrations to store the people information such as
first and last name, and their email address.

Adds a new route /api/registration that creates a new registration on
the database.

Add documentation and test for this new API route.

STORY: TG-152

Registration being the process of someone registering with their
email to receive updates, notifications or news on the BioPocket project.

Adds a new table `registrations` to store the people information such as
first and last name, and their email address.

Adds a new route `/api/registration` that creates a new registration on
the database.

Add documentation and test for this new API route.

STORY: TG-152
@coveralls
Copy link

coveralls commented Sep 13, 2018

Coverage Status

Coverage increased (+0.4%) to 92.971% when pulling 88b2309 on update-subscription into be63da9 on sprint-2-discovering-actions.

Mathias Oberson and others added 11 commits September 27, 2018 17:26
This route will allow the mobile app to test for the availability of an
email address while a user type it in the registration form.

Add documentation for this new route, along with a couple of unit test.
Add missing `@module` decoractor on some functions.

Move the `getExpected...` functions from the tests to the test utilities
Add test and documentation for this new route.
Also add a new varible in the sample file to limit pagination.
It should be limited to 25 (maybe it ought to be the default value?)
server/api/registrations/registrations.api.raml Outdated Show resolved Hide resolved
server/api/registrations/registrations.api.spec.js Outdated Show resolved Hide resolved
server/api/registrations/registrations.policy.js Outdated Show resolved Hide resolved
server/api/registrations/registrations.policy.js Outdated Show resolved Hide resolved
AlphaHydrae and others added 2 commits March 26, 2019 15:35
Co-Authored-By: Tazaf <Tazaf@users.noreply.github.com>
@AlphaHydrae AlphaHydrae merged commit 5cce158 into sprint-2-discovering-actions Mar 26, 2019
@AlphaHydrae AlphaHydrae deleted the update-subscription branch March 27, 2019 08:33
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.

None yet

3 participants