Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • C Comunidade Universitaria Front
  • Project information
    • Project information
    • Activity
    • Labels
    • Planning hierarchy
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Infrastructure Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Comunidade Universitaria
  • Comunidade Universitaria Front
  • Merge requests
  • !24

Closed
Created Oct 08, 2023 by Filipe de Oliveira de Freitas@filipe.freitasOwner
  • Report abuse
Report abuse

Melhorias

  • Overview 0
  • Commits 5
  • Changes 8

Esse PR não tem o intuito de ser Mergeado. Ele apenas evidência alguns problemas que encontrei ao rodar o Frontend na Web.

  1. a4422598 - fix: missing imports or imports problems ✅

    Galera, estamos esquecendo de salvar as dependências, estamos apenas instalando elas. Estamos copiando e colando o comando do site do NPM que não vem com o save.

    MERGE JÁ REALIZADO EM !25 (merged)

  2. 18f8788b - fix: Cannot use Android Toast. Wont work in Web 🚧

    Acho que isso já foi corrigido na branch https://tools.ages.pucrs.br/comunidade-universitaria/comunidade-universitaria-front/-/tree/feature/01-screen-pesquisa-de-evento pois esse file não existe mains

  3. e66d4816 - fix: wrong usage of event spot and address ✅

    Uso invertido

    Spot = É a descrição resumida como "Auditorio Araujo Vianna"

    Address = Endereço completo como "Parque Farroupilha, 685 - Farroupilha, Porto Alegre - RS, 90035-191"

    MERGE JÁ REALIZADO EM !25 (merged)

  4. b35d8608 - feat: date picker 🚧

    Nosso campo de data e horário, na criação do Evento aceita TEXTO! Deveríamos no mínimo criar uma máscara para aceitar corretamente os valores. O frontend nem informa erro de conteúdo do texto, só falha a rota.

  5. 0ef4fe72 - feat: routes are good 🚧

    Deveríamos usar rotas, deveríamos nos importar com a Web. Fora que temos um campo de Share de evento, precisamos compartilhar um link que aponte para a página no Web, para isso existem Deep Link

Edited Oct 08, 2023 by Filipe de Oliveira de Freitas
Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: melhorias