Skip to content
This repository was archived by the owner on Mar 13, 2025. It is now read-only.

[$150] Project UBahn | Catalog APIs #639

Closed
wdprice opened this issue Sep 2, 2020 · 11 comments
Closed

[$150] Project UBahn | Catalog APIs #639

wdprice opened this issue Sep 2, 2020 · 11 comments

Comments

@wdprice
Copy link

wdprice commented Sep 2, 2020

Catalog all v5 APIs developed for u-bahn and ensure adherence to v5 spec.

@wdprice wdprice added the enhancement New feature or request label Sep 2, 2020
@wdprice wdprice added this to the v1.01 - Stability Release milestone Sep 2, 2020
@callmekatootie
Copy link
Collaborator

What does catalog refer to here - we have the swagger specification - does that qualify - https://github.com/topcoder-platform/u-bahn-api/blob/develop/docs/swagger.yaml

@callmekatootie
Copy link
Collaborator

@wdprice ^

@callmekatootie
Copy link
Collaborator

API being audited: topcoder-archive/topcoder-platform-u-bahn-api#48

@callmekatootie
Copy link
Collaborator

Issues observed being resolved - topcoder-archive/topcoder-platform-u-bahn-api#50

@callmekatootie
Copy link
Collaborator

API code base completed. Now auditing apis in this repository itself...

@callmekatootie
Copy link
Collaborator

callmekatootie commented Oct 8, 2020

  • Templates endpoint
    issues:
    Missing head /templates/:id => VALID issue
    warn:
    Missing get /templates => VALID issue. Need an endpoint to know which templates have been created

@callmekatootie
Copy link
Collaborator

callmekatootie commented Oct 8, 2020

  • Uploads endpoint
    issues:
    • Missing head /uploads/:id => VALID issue
    • get /uploads does not support paging => NOT a valid issue. We are using dynamodb and thus pagination is not supported.
      warn:
      /uploads/:id has a PATCH method but not PUT => NOT a valid issue. Not expecting updates to all fields - only couple of them

@callmekatootie
Copy link
Collaborator

callmekatootie commented Oct 8, 2020

  • Skills endpoint
    issues:
    get /skills does not support paging
    warn:
    There is no skills related code under src.

=> Remove all references to this endpoint

@callmekatootie callmekatootie changed the title Catalog APIs [$150] Project UBahn | Catalog APIs Oct 9, 2020
@cwdcwd
Copy link
Contributor

cwdcwd commented Oct 9, 2020

Contest https://www.topcoder.com/challenges/30144966 has been created for this ticket.

This is an automated message for lazybaer via Topcoder X

@cwdcwd
Copy link
Contributor

cwdcwd commented Oct 9, 2020

Contest https://www.topcoder.com/challenges/30144966 has been updated - it has been assigned to xxcxy.

This is an automated message for lazybaer via Topcoder X

@cwdcwd
Copy link
Contributor

cwdcwd commented Oct 9, 2020

Payment task has been updated: https://software.topcoder.com/review/actions/ViewProjectDetails?pid=30144966

This is an automated message for lazybaer via Topcoder X

@cwdcwd cwdcwd added the tcx_Paid label Oct 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants