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.

Sign up for GitHub

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

Jump to bottom

Finishes #13 - supporting organization enrolment and validation #17

Merged
merged 15 commits into from
Feb 28, 2018
Merged

Finishes #13 - supporting organization enrolment and validation #17

merged 15 commits into from
Feb 28, 2018

Conversation

ortonomy
Copy link
Owner

app.js

  • adds new app.js endpoint /orgaccess to api with email side effects
  • uses plugin to modify responses and redact sensitive information
    schema
  • adds new functions to enable org enrolment, validation
  • adds new table to track org access requests

Use when testing ortonomy/flingapp-frontend#49

ortonomy added 15 commits February 10, 2018 13:08
issue #13 - removes requirement for new org to have a domain
04a91dc
removes dependency on inserting domain to organization table in fixtu…
eb08dc2 
…re data
issue #13 - removes NOT NULL requirement on organization.org_domain
ee99b96
issue #13 - create organization schema modifications
bf5d4d3 
Issue related
- [x] fixes permissions for INSERT for create_organization mutation
- [x] adds second user without organization to seed data for testing org creation

Misc. (bad practice)
- [x] fixes mapping issue between project and text note map
issue #13 - adds tables and functions for managing org access requests
0050a53
issue #13 - simplifies update_user_by_id and add not found check and …
fabff37 
…adds org to parameters
rewrites update user by ID to not use unupdateable view
a9d6865
issue #13 - finishes update user by id. tested on local machine
dde5265
issue #13 - new update user by email
c1e2105
issue #13 - finishes request and validate Org functions
f4cc49b
issue #13 - adds comments for self-documenting API to views
9022e16
issue #13: fixes mistake that means admin details were not returned
0ad7ce7
issue #13 - adds email support for requesting org access
6eed14f 
- [x] new DB types to support org access request return type
- [x] updated support functions for custom org inserts
- [x] new app.js proxy endpoint /orgaccess for accepting requests for org access
- [x] side effects of using /orgaccess will send emails for any requestAccessToOrg mutations
issue #13 - rewrites redacted API response on proxies for efficiency
e16a700
finishes #13 - finishes support for org enrolment
5113cec 
- adds new API endpoint /orgaccess
- emails will be sent for validation requests when requests received at this endpoint
- adds new function to get org_access_request without being owner: ``org_request_by_requestor_id``
@ortonomy ortonomy added this to the Version 1.0 milestone Feb 27, 2018
@ortonomy ortonomy requested a review from Jskobos February 27, 2018 06:41
@ortonomy ortonomy mentioned this pull request Feb 27, 2018
Open
7 tasks
@ortonomy ortonomy merged commit 01346d2 into development Feb 28, 2018
@ortonomy ortonomy deleted the issue13 branch February 28, 2018 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jskobos Jskobos Awaiting requested review from Jskobos

Assignees
No one assigned
Labels
feature priority: blocker progress: testing
Projects
None yet
Milestone
Version 1.0
Development

Successfully merging this pull request may close these issues.
1 participant
@ortonomy