tallerify / shared-server Goto Github PK

19/32 tests fail while running the suite with istanbul 😕

Investigate if it's possible and easy to visualize reports in Travis or in another online service.

http://rebilly.github.io/ReDoc/?url=https://gist.githubusercontent.com/agrojas/ea2001512de94ce03ebebef797a6cdd0/raw/537e016ee678ebb15177920f990ea59040061068/music-shared-api.yaml#

Some changes were made to the endpoints but where never updated.

Right now they are always returning OK, and I don't know why

As described here

I'm planning in following this guide, https://devcenter.heroku.com/articles/mean-apps-restful-api#see-the-sample-application-running

No hay endpoints para la actividad reciente del usuario. De momento simulo que son las ultimas CANCIONES escuchadas (porque pinto). Si les parece otra cosa, hagamos otra cosa, sino simplemente trackeen (trackeen != cancion. anger intensifies) las canciones que el usuario va escuchando y es una lista de las N ultimas ? Lo que les sea mas facil, despues diganme como quedo por fa 😄

Merge final code into master and tag it.

Before doing that:

• Ensure test are ok both in Travis and local.
• Check that coverage is above 75%
• Check that app works both with and without Docker.

After doing that:

• Check that release is properly deployed to Heroku and freeze deployments to it until the checkpoint finishes.

https://lbroudoux.wordpress.com/2016/07/05/dockerize-your-mean-application/

Estan mal documentados en ese Rebilly. Que devuelvan Song y no SongShort al hacer un get a las song de una playlist/album/artist

Esta relacionado con #58

This way we could see the coverage of the TypeScript files rather thant the traspiled ones.

http://rebilly.github.io/ReDoc/?url=https://gist.githubusercontent.com/agrojas/ea2001512de94ce03ebebef797a6cdd0/raw/537e016ee678ebb15177920f990ea59040061068/music-shared-api.yaml#

Currently, any string is valid for fields like user.birthdate

• GET /users/me
• PUT /users/me
• GET /users/me/contacts

Doc

• UserShort tenga el campo images[]
• ArtistShort tenga el campo images[]
• AlbumShort tenga el campo images[]
• Playlist tenga el campo images[] -> De momento no tiene imagenes, podemos hacer que tenga de images las de la primer cancion o album o cualquier gilada ? Asi tengo como mostrarla

SongShort no lo necesito, pero si les sirve por cohesividad atroden

sooooo muuuuch code repetition

Los tenia como

POST/DELETE a: users/{my_id}/follow
Body:

{
  "userId": "user_to_follow/unfollow_id"
}

Dado que ya es authenticated, podemos sacar el body y que sea la url
users/{user_to_follow/unfollow_id}/follow ? De esta forma no mandariamos mi id ya que lo obtienen del token

Porque el cliente nativo de android no permite Deletes con body (es un bug de ellos, esta fileado)

Yapa: Asi como propongo estan los follow/unfollow a artistas y canciones en la docu, supongo que les quedo mal en estos endpoints justo jaja

Lo que dice el titulo.. Artist y playlist tienen un endpoint para obtener las canciones de cada uno, album no.

Talves los agreguen despues, por las dudas marco los que faltan al dia de hoy

• Trending artists (artists/trending?) -> Lo hice sin auth, si llega a necesitar auth avisen!
• Trending songs (songs/trending?) -> Lo hice sin auth, si llega a necesitar auth avisen!
  - [ ] Playlists de un user (me/playlists?)
• Follow/Unfollow user (users/{id}/follow POST/DELETE?) -> Tampoco sabemos si es doble relacion o simple, si es doble aca se cambiaria por algun request/accept/decline ?

http://tostring.it/2014/06/23/advanced-logging-with-nodejs/

Change IDE skin for enhacing the developer instead of writing garbage

FINISH HIM! (sequelize) 😬

Currently it's installed but it doesn't run as far as I'm aware of.