Description

266

Questions

It seems sketchy to be overwriting the Settings schema directly from the server func, but there's no existing server func for adding emails to the Settings schema. It also seems sketchy to just pass the request directly into the server func, though that's what the issue said to do. Should we validate that the user ids correspond to actual users first?

Type of change

[x] New feature (non-breaking change which adds functionality)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have assigned reviewers to this PR

To test

{
    "userEmails": ["ethanhmaness@gmail.com", "emaness@vols.utk.edu"],
    "adminIds": ["6625acc38b667741185a92ae", "662921e0e3a3ebfe912a9200"]
}

Send it a second time to check that it doesn't add duplicate emails.

NOTE: You can't use Postman for this, gotta do it directly thru the browser so your account can be authenticated.

hack4impact-utk / compassion-ministries

266 create server funcapi endpoint for updating allowed users #346

Description

266

Questions

Type of change

Checklist:

To test