Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Adicionados módulos de transportes e viagens #129

Open
wants to merge 36 commits into
base: develop
Choose a base branch
from
Open

Adicionados módulos de transportes e viagens #129

wants to merge 36 commits into from

Conversation

vitor-msp
Copy link

Issue relacionada: #97 "Escalar o sistema para permitir o transporte de suprimentos aos abrigos"

mod-transports-trips

@vitor-msp
feat: added prisma schema and migrations
f44dde7
@vitor-msp
feat: added transports module structure
d079fcf
@vitor-msp
feat: added endpoint post transport
82a7acc
@vitor-msp
feat: added endpoint put transport
4046efa
@vitor-msp
feat: added transport managers module structure
07ecfcd
@vitor-msp
feat: added endpoint post transport managers
6814988
@vitor-msp
feat: added trips module structure
c87943c
@vitor-msp
fix: changed name format of the some trip fields
461a79c
@vitor-msp
feat: added endpoint post trip
52fba27
@vitor-msp
feat: added endpoint put trip
cbe39e9
@vitor-msp
feat: added field canceled in trips table
2c3aa1a
@vitor-msp
feat: added endpoint delete trip
2a275cd
@vitor-msp
feat: added guards
a597811
@vitor-msp
feat: added user validation in the trips module
64bd391
@vitor-msp
refactor: improved error messages
c023337
@vitor-msp
feat: added endpoint get transports
325f732
@vitor-msp
feat: added endpoint get transport by id
e9641e0
@vitor-msp
feat: added endpoint get trip by id
d2a40e8
@vitor-msp
feat: added endpoint get trips
9985a85
@vitor-msp
feat: changed departure_datetime field in trip to datetime
96140c9
@vitor-msp
feat: trip search changed to use departure datetime range
b8f774b
@vitor-msp
feat: added field departure_neighborhood in trips table
9118664
@vitor-msp
feat: added endpoint get trip cities
b485816
@vitor-msp
feat: added field departure_state in trips table
0c24403
@vitor-msp
feat: added endpoint get trip states
4d49f92
@vitor-msp
feat: added departure_state in the get trips filter
1514c82
@vitor-msp
Merge pull request #1 from SOS-RS/develop
0fb9add 
sync my repo
@vitor-msp
Merge branch 'develop' into feat/transports
102cc76
@vitor-msp vitor-msp mentioned this pull request May 16, 2024
Open
@HelioDantas
Copy link

Issue relacionada: #97 "Escalar o sistema para permitir o transporte de suprimentos aos abrigos"

mod-transports-trips

bem legal.
Podemos inclui a posterior os itens e quantidade que essa viagem está levando

rodrigooler
rodrigooler requested changes May 17, 2024
View reviewed changes
Copy link
Contributor

@rodrigooler rodrigooler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Revisitei o conteúdo e está excelente! Tenho apenas algumas sugestões de aprimoramento especificamente relacionadas aos Exception Filters.

src/transport-managers/transport-managers.service.ts Outdated Show resolved Hide resolved
src/transport-managers/transport-managers.service.ts Outdated Show resolved Hide resolved
src/transports/transports.service.ts Outdated Show resolved Hide resolved
src/trips/TripsDao.ts Outdated Show resolved Hide resolved
src/trips/TripsDao.ts Outdated Show resolved Hide resolved
src/trips/TripsDao.ts Outdated Show resolved Hide resolved
src/trips/trips.service.ts Outdated Show resolved Hide resolved
src/guards/transport-manager.guard.ts Outdated Show resolved Hide resolved
@rodrigooler rodrigooler added enhancement New feature or request can-merge labels May 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rodrigooler rodrigooler rodrigooler approved these changes

@marcieltorres marcieltorres Awaiting requested review from marcieltorres

Assignees

@vitor-msp vitor-msp

Labels
can-merge enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vitor-msp @HelioDantas @marcieltorres @rodrigooler