Comments (13)
@maxceem can you pls take a look this.
from taas-apis.
I had a look, understood the issue, and would update with details tomorrow.
from taas-apis.
Internally GET /taas-teams
endpoint calls Projects API GET /projects
.
Projects API support sorting, pagination, and filtering by name: https://api.topcoder-dev.com/v5/projects/?sort=lastActivityAt+desc&perPage=20&page=1&name=*test*
.
-
GET /taas-teams
should support pagination paramspage
andperPage
(pass to Projects API).-
TaaS app should set pagination headers the same way as for other TaaS endpoints. Projects API returns pagination headers:
-
Replace the custom code https://github.com/topcoder-platform/taas-apis/blob/feature/integration-test-fix/src/common/helper.js#L128-L134, with CORS config https://github.com/topcoder-platform/resources-api/pull/45/files
-
-
GET /taas-teams
should support sorting paramssortBy
andsortOrder
and pass them to Projects API in one paramsort
assort=<sortBy>+<sortOrder>
-
GET /taas-teams
should support filtering paramname
and pass it to Projects API -
Make sure that all these params work as per the description above and I didn't miss anything.
-
Update API documentation: Swagger, Potman.
FYI @nkumar-topcoder
from taas-apis.
@imcaizheng you may pick it up if it looks good for you.
from taas-apis.
PR created #44
from taas-apis.
Contest https://www.topcoder.com/challenges/30156198 has been created for this ticket.This is an automated message for maxceem via Topcoder X
from taas-apis.
Contest https://www.topcoder.com/challenges/30156198 has been updated - it has been assigned to aaron2017.This is an automated message for maxceem via Topcoder X
from taas-apis.
Works good.
The only thing, I guess we shouldn't add result
when we return arrays in the response because as per swagger we have to return arrays of records directly, like:
[
{},
...
{}
]
instead of
{
result: [
{},
...
{}
]
}
So I removed result
for all the "list" endpoints f3328f7
@imcaizheng please, let me know if I missed something, and we should handle it another way, rather than removing result
.
from taas-apis.
@maxceem other operations such as GET /jobs
, GET /jobCandidates
and GET /resourceBookings
return data with the result
field, I think it may make API clients a little happy by letting GET /taas-teams
to return data the same way, also considering the taas-teams endpoints were developed after the jobs, jobCandidates and resourceBookings endpionts. I would prefer to change the swagger spec instead if it is not too late to do that.
from taas-apis.
Thanks, @imcaizheng.
@nkumar-topcoder as per swagger and implementation all list
endpoints return data in result
field:
While endpoints which return single records, return data directly without result
field:
Also, other Topcoder V5 API return all the data without result
field in the response. For example Challenges API:
Do you know if there was some intention behind adding result
for the "list" endpoints or this happened unintentionally?
Also, I think Topcoder API V5 guidelines require to return data directly.
from taas-apis.
@maxceem
not sure , @veshu can help us here. pls reach out to him (offline on slack).
feel free to implement as deemed fit. thanks
from taas-apis.
Ok, I believe there should not be result
in responses, so I've removed it in endpoints and in Swagger.
from taas-apis.
Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30156198This is an automated message for maxceem via Topcoder X
from taas-apis.
Related Issues (20)
- [$100] Some notifications are not received HOT 33
- [$50] Replace webhook which listens to created events HOT 4
- Interview Improvements - Summary
- [$75] Fix updating interview method on webhooks HOT 14
- [$75] Use `metadata` to identify Nylas update events instead of parsing `description` HOT 2
- Issue when connecting Google/Microsoft calendar and then removing it HOT 12
- Interview scheduled email is not sent
- [$50] Add authentication for a new Nylas Page webhook. HOT 5
- [$80] Send interview "canceled" and interview "rescheduled" email notifications HOT 12
- [$30] Use existent env variable during DB migration HOT 5
- Some interviews are stuck in scheduling, therefore unable to cancel/reschedule them HOT 3
- Attendees are not listed in the Interview reminder(customer), Interview complete(customer) and Reminder: Take action to reserve your talent(customer) emails HOT 2
- [$40] The meeting link sent to the JC in the 'Reminder: Interview coming up' email says 'You do not have permissions to perform this action' HOT 14
- Add a new `Did not Post` rcrmStatus to the taas-api HOT 1
- Migration Open Jobs which are only present on Gigs Listing MFE
- The email notification 'before 1 hour of interview time' and the 'Post Interview Action Reminder' notification are not received HOT 1
- [Prod] Interview confirmation email notification is not sent sporadically HOT 4
- Download resume email was not received HOT 2
- Cancel interview doesn't work - Interview is not getting canceled and no emails sent HOT 3
- Internal server error while trying to schedule an interview
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from taas-apis.