Skip to content

397 troca card de tempo#399

Merged
GRHInvDev merged 2 commits into
mainfrom
397-troca-card-de-tempo
Jul 6, 2026
Merged

397 troca card de tempo#399
GRHInvDev merged 2 commits into
mainfrom
397-troca-card-de-tempo

Conversation

@GRHInvDev

@GRHInvDev GRHInvDev commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • New Features

    • Added sector management in the admin area, including creating, editing, activating/deactivating, and deleting sectors.
    • Sector options are now loaded dynamically across user management and activity history, improving consistency.
  • Bug Fixes

    • Improved company/user filtering so results respect the selected company more reliably.
    • Replaced a fixed mobile carousel slide with a weather widget for a more relevant dashboard experience.
  • Chores

    • Bumped the app version to 1.52.0.

rbxyz and others added 2 commits June 29, 2026 15:28
O video "Valores Grupo RHenz" (iSdz3gxUpAI) ja vem do array itens,
mas havia um CarouselItem hardcoded com o mesmo embed, fazendo o video
aparecer duas vezes apenas no mobile. Removido o item duplicado; o
WeatherWidget (exclusivo do mobile) foi mantido.

Bump 1.51.1 -> 1.51.2 (patch).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
- Novo model Setor (lista gerenciável); User.setor segue como string
- Migration cria tabela setores e insere os setores já existentes
  (pré-definidos + valores distintos em users.setor)
- Router setores (list/create/update/delete) gateado por
  sudo || can_manage_dados_basicos_users
- Modal "Gerenciar Setores" na tela de gestão de usuários
- Filtro de Empresa passa a usar o cadastro real (Empresa) + Filial
  em cascata; listUsers ganha filtro empresaId
- Dropdowns/labels de setor passam a ler da tabela Setor
- Versão 1.51.2 -> 1.52.0 (minor)

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
elo Building Building Preview, Comment Jul 6, 2026 2:04pm

@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 93c0a014-0a28-4abb-9f65-1802b2e62f98

📥 Commits

Reviewing files that changed from the base of the PR and between 9322978 and 2e86cdd.

📒 Files selected for processing (9)
  • package.json
  • prisma/migrations/20260706120000_create_setores/migration.sql
  • prisma/schema.prisma
  • src/app/(authenticated)/admin/users/_components/setores-dialog.tsx
  • src/app/(authenticated)/admin/users/page.tsx
  • src/components/dashboard/videos-carousel.tsx
  • src/server/api/root.ts
  • src/server/api/routers/setores.ts
  • src/server/api/routers/user.ts

📝 Walkthrough

Walkthrough

This PR introduces a "Setor" (sector) management feature: a new Prisma model and migration, a setoresRouter TRPC API with CRUD and permission checks, and a SetoresDialog admin UI component. The admin users page is updated to use dynamic setores instead of hardcoded lists, and listUsers gains an empresaId filter. Unrelated changes include a package version bump and a carousel widget swap.

Changes

Dynamic Setores Feature

Layer / File(s) Summary
Setor schema and migration
prisma/schema.prisma, prisma/migrations/20260706120000_create_setores/migration.sql
Adds Setor model with unique name/value fields and migration creating setores table with unique constraints, seed data, and backfill from existing users.setor.
setoresRouter CRUD API
src/server/api/routers/setores.ts, src/server/api/root.ts
Implements permission-checked list, create, update, delete procedures with value normalization and duplicate/linked-user checks; registers router in appRouter.
SetoresDialog management UI
src/app/(authenticated)/admin/users/_components/setores-dialog.tsx
New client component for creating, editing, activating/deactivating, and deleting sectors via TRPC mutations with toast feedback and delete confirmation.
Admin users page integration
src/app/(authenticated)/admin/users/page.tsx
Replaces hardcoded sector lists with dynamic setores data across filters, user rows, editor selects, and audit formatting; renders SetoresDialog.
listUsers empresaId filter
src/server/api/routers/user.ts
Adds optional empresaId input filtering users via their associated filial.

Estimated code review effort: 3 (Moderate) | ~30 minutes

Unrelated Misc Updates

Layer / File(s) Summary
Version bump and carousel widget swap
package.json, src/components/dashboard/videos-carousel.tsx
Bumps package version to 1.52.0; replaces a hardcoded YouTube iframe slide with a WeatherWidget slide in the mobile carousel.

Sequence Diagram(s)

sequenceDiagram
  participant Admin
  participant SetoresDialog
  participant setoresRouter
  participant Database

  Admin->>SetoresDialog: open dialog, add/edit/delete setor
  SetoresDialog->>setoresRouter: create/update/delete mutation
  setoresRouter->>setoresRouter: checkPermission (sudo/can_manage_dados_basicos_users)
  setoresRouter->>Database: validate uniqueness, check linked users
  Database-->>setoresRouter: query result
  setoresRouter-->>SetoresDialog: success or TRPCError
  SetoresDialog->>SetoresDialog: invalidate setores.list, user.listUsers
  SetoresDialog-->>Admin: toast feedback
Loading

Possibly related PRs

  • GRHInvDev/elo#269: Both PRs modify the admin users page and userRouter.listUsers filter inputs/logic for querying and rendering the user list.
  • GRHInvDev/elo#300: Directly related change to videos-carousel.tsx adding a WeatherWidget slide with the enterprise prop.
  • GRHInvDev/elo#381: Both PRs modify the admin Users page's "Movimentações (auditoria)" flow, with this PR extending it to render setor labels from the new dynamic list.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 397-troca-card-de-tempo

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@GRHInvDev
GRHInvDev merged commit 3cf761f into main Jul 6, 2026
6 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants