feature: fetch companies

thgmhz / bus-promise

:oncoming_bus: Biblioteca que busca informações em tempo real da frota de ônibus da SPTrans na cidade de São Paulo.

MIT License

61 stars 11 forks source link

feature: fetch companies #44

Closed itsdaiego closed 7 years ago

itsdaiego commented 7 years ago

O que este PR faz?

Adiciona o tipo "companies" que retorna a lista de empresas por área de operação:

Precisa alterar a documentação?

[x] Sim. Descreva as funcionalidades que serão afetadas com esta mudança.
[ ] Não.

Checklist do Desenvolvedor

[x] Atualizou o build de browser
[x] Atualizou o build de server

Checklist do Admin

[x] Versão atualizada no package.json
[x] CHANGELOG.md atualizado com as mudanças
[x] bus-promise atualizado no NPM

itsdaiego commented 7 years ago

@thiagommedeiros done! :+1:

thgmhz commented 7 years ago

@drodrigo Estava pensando, essa estrutura de dados que a SPTrans retorna é horrível, acho que podemos melhorar isso e tornar a vida do desenvolvedor mais feliz, heheheh.

Ao invés disso:

{
  "hour": "12:51",
  "companiesByOperationArea": [
    {
      "operationAreaCode": 1,
      "companies": [
        {
          "operationAreaCode": 1,
          "referenceCode": 33,
          "name": "GATO PRETO"
        }
      ]
    },
    {
      "operationAreaCode": 2,
      "companies": [
        {
          "operationAreaCode": 2,
          "referenceCode": 34,
          "name": "SANTA BRIGITA"
        }
      ]
    }
  ]
}

Poderíamos fazer isso:

{
  hour: "12:51",
  companies: [{
    operationAreaCode: 1,
    referenceCode: 33,
    name: "GATO PRETO"
  },
  {
    operationAreaCode: 2,
    referenceCode: 34,
    name: "SANTA BRIGIDA"
  }]
}

O que vc acha?

itsdaiego commented 7 years ago

Gostei!! fica bem melhor, farei! haha

itsdaiego commented 7 years ago

@thiagommedeiros fiz as alterações, sobre o refactoring ficou parecido com a sua ideia, da uma olhada e me diga oq ach a =) (não sei se essa é a forma mais "enxuta" de resolver o problema)

itsdaiego commented 7 years ago

Show!!! Tem algo a mais a se fazer antes de mergear @thiagommedeiros ?