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

feat: enable create shelters #53

Merged
merged 36 commits into from
Jun 4, 2024
Merged

feat: enable create shelters #53

merged 36 commits into from
Jun 4, 2024

Conversation

willMoraes
Copy link
Collaborator

@willMoraes willMoraes commented May 16, 2024 •
edited
Loading

Closes #37

  • Enable user create more than one shelter
  • Enable user to delete shelter by Id
  • Create User Shelters page
  • Create Edit Shelter page
  • Crete delete shelter dialog

image

@vercel Vercel
Copy link

vercel bot commented May 16, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
sos-pet ⬜️ Ignored (Inspect) Visit Preview Jun 3, 2024 11:55pm

@willMoraes willMoraes self-assigned this May 17, 2024
@emiliosheinz
Copy link
Owner

Revisei a questão da estrutura de rotas como tu pediu e acho que da pra melhorar um pouco principalmente pra facilitar a questão de controle de autenticação depois.

Acredito que tudo que é coisa referente ao usuário em si pode ficar dentro de /user dessa forma conseguimos colocar um auth guard nessa camada.

Ficaria algo como:

src
  app
    about
    api
    privacy-policy
    signin
    terms-of-use
    shelters (seria para uma possivel listagem public)
    user (tudo que é referente a um user)
      shelters (listagem de shelter do usuário)
        create (criação de um shelter para o usuário)
        [id]/edit (edição de shelter do usuário)

O que tu acha?

@willMoraes willMoraes force-pushed the feat/enable-create-shelters branch from 051b6dc to f935cf5 Compare May 28, 2024 00:40
@willMoraes willMoraes marked this pull request as ready for review May 28, 2024 00:49
emiliosheinz
emiliosheinz requested changes May 29, 2024
View reviewed changes
src/app/user/shelters/columns.tsx Outdated Show resolved Hide resolved
src/app/user/shelters/[id]/edit/layout.tsx Outdated Show resolved Hide resolved
src/app/user/shelters/[id]/edit/page.tsx Outdated Show resolved Hide resolved
src/app/user/shelters/columns.tsx Outdated Show resolved Hide resolved
src/app/user/shelters/create/page.tsx Outdated Show resolved Hide resolved
src/components/shelter/form-edit-register/index.tsx Outdated Show resolved Hide resolved
src/schemas/shelter.ts Outdated Show resolved Hide resolved
src/server/api/routers/shelter.ts Outdated Show resolved Hide resolved
src/server/api/routers/shelter.ts Outdated Show resolved Hide resolved
src/server/api/routers/shelter.ts Outdated Show resolved Hide resolved
@emiliosheinz
Copy link
Owner

Só deixando esse comentário pra lembrar que tem a questão do UUID pra ser vista ainda. Correto?

willMoraes added 21 commits June 3, 2024 19:53
@willMoraes
feat: ✨ clean shelter page to enable creating more than one shelter p…
aa0a5a7 
…er user
@willMoraes
feat: ✨ add table component
0ba1788
@willMoraes
feat: ✨ add shelter findById method in routes
e178312
@willMoraes
feat: ✨ create userShelters query and route
03a0169
@willMoraes
feat: ✨ create user shelters page
e4823c0
@willMoraes
feat: ✨ create edit shelter page
e621534
@willMoraes
refactor: ♻️ create formEditRegister component to be used in both pages
bc808aa
@willMoraes
feat: ✨ add shelters not found message in user shelters page
cfd96df
@willMoraes
feat: ✨ add dialog component
074da0a
@willMoraes
feat: ✨ create delete shelter dialog
b0bb466
@willMoraes
refactor: ♻️ move /edit route to user routes
1c62006
@willMoraes
refactor: ♻️ update /edit shelter links
291ef33
@willMoraes
feat: ✨ throw not found error when has no user shelter
cbbf2a1
@willMoraes
feat: ✨ add skeleton to edit and shelters pages
a08712c
@willMoraes
feat: ✨ add coords to form component
8d19945
@willMoraes
refactor: ♻️ update api methods names
065973f
@willMoraes
fix: 💄 adjust mobile shelters table
90d6fda
@willMoraes
refactor: ♻️ move data-table component to folder
6766ea2
@willMoraes
fix: 🐛 improve user redirect when try to edit without session
3c06266
@willMoraes
refactor: ♻️ improve shelters list table folders structure
e3148a0
@willMoraes
refactor: ♻️ rename table column to capacity to fit pattern
7a633b5
willMoraes added 14 commits June 3, 2024 19:53
@willMoraes
refactor: ♻️ remove unecessary layouts from edit and create shelter
23997f8
@willMoraes
feat: ✨ change redirect to shelters list
88002d0
@willMoraes
refactor: ♻️ rename edit and create shelters page
036391f
@willMoraes
refactor: ♻️ improve items validations with optional chaining
11938f1
@willMoraes
refactor: ♻️ improve userShelters findAll type
29b3acb
@willMoraes
refactor: ♻️ move form-edit-register component to app folder
8c6766e
@willMoraes
refactor: ♻️ destructu input props
70221a7
@willMoraes
refactor: ♻️ move user shelters procedures
5985fea
@willMoraes
fix: 🐛 adjust shelters types to edit and create shelter
2ebecdf
@willMoraes
fix: 🐛 adjust shelter type in ShelterTable
8a0ed4c
@willMoraes
feat: ✨ send user to shelters list when finishing create
6660547
@willMoraes
feat: ✨ add alert to shelter edit page, when some error occurs
37cc727
@willMoraes
feat: ✨ add migration to turn uuid required
c25e955
@willMoraes
feat: ✨ update edit route to use uuid instead id
1c688fc
@willMoraes willMoraes force-pushed the feat/enable-create-shelters branch from 8918eac to 1c688fc Compare June 3, 2024 23:48
@emiliosheinz emiliosheinz merged commit 30cf29c into dev Jun 4, 2024
3 checks passed
@emiliosheinz emiliosheinz deleted the feat/enable-create-shelters branch June 4, 2024 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@emiliosheinz emiliosheinz emiliosheinz approved these changes

Assignees

@willMoraes willMoraes

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Poder cadastrar mais de um abrigo por usuário
2 participants
@willMoraes @emiliosheinz