Skip to content

Production release 2.4.16 #427

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

Merged
merged 38 commits into from
Dec 17, 2019
Merged

Production release 2.4.16 #427

merged 38 commits into from
Dec 17, 2019

Conversation

vikasrohit
Copy link

Supporting release for connect 2.4.16 with:

  • Default status of projects now would be In Review
  • Added new endpoints related to the members and invite use cases

r0hit-gupta and others added 30 commits September 19, 2019 23:56
Cannot reproduce this issues locally, so this fix is a guess.
Project has been indexed in a wrong way for unit test.
Also, we have to wait until the index is refreshed to make tests consistent.
Added endpoint for getting project members with ability to get additional data form member service and pre-populate the member list.

ref: Winner submission from challenge 30107353 - "Topcoder Project Service - Project Members Traits"
Also, removed disabling eslint rule.
Added/updated following endpoints to get user information with additional data:
1. project member by id
2. list of all invites inside the project
3. authenticated user invite details
1. Return only single object for invite endpoint
2. Handle 404 error for get project member by id endpoint
3. Return only pending project member invites
feat: project member & invites endpoints with additional fields
Mask invitation emails with asterisks #3388
- properly ignore invites which don't have userId
- don't fail populating details for all the members if just for some members we fail to load traits. Just log error and debug information in such case
- don't set 'null' for photoURL if we didn't get it, so we keep the exact falsy value we got from the server
@RishiRajSahu RishiRajSahu merged commit 2d32e94 into master Dec 17, 2019
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.

6 participants