Conversation
RafaMelazzo
requested review from
Mario-Junior,
camiladiasds,
Copilot,
devandremachado,
fabiano-mallmann and
kevincorsigon
|
Por favor, catalogue-o seguindo as instruções nesta documentação. [Via VPN]. 💁 Qualquer problema ou dúvida, estamos no Slack, basta abrir um ticket no canal #help-foundation-platform. |
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| abstract class AbstractPoiTypeEnums | ||
| { | ||
| const POS = 'pos'; |
There was a problem hiding this comment.
Dúvida: aqui não deveria ser PascalCase?
There was a problem hiding this comment.
Eu lembro da gente ter questionado isso, e disseram que não tinha um padrão ainda. Mas isso foi bem no começo, já tem uma definição? Na documentação ainda está da forma como inseri no arquivo.
De qualquer forma, como não tinha certeza de como essa informação vai chegar, utilizei strtolower para não dar erro na comparação, mas no momento de armazenar a informação, ela é salva da forma como chegar pra gente.
Important
Certifique-se de criar o PR para a branch develop.
Tarefa: ECPJ-473
Qual o tipo de PR é esse? (marque todos os aplicáveis)
Descrição
poiTypeà classe Configuration para suportar o modelo One Stone;AbstractPoiTypeEnumspara validação de valorespoiTypepermitidos;