Skip to content

Cleanup - #41

Open
flaviomir2712-hue wants to merge 2 commits into
4GeeksAcademy:mainfrom
marveen-riskwait:main
Open

Cleanup#41
flaviomir2712-hue wants to merge 2 commits into
4GeeksAcademy:mainfrom
marveen-riskwait:main

Conversation

@flaviomir2712-hue

Copy link
Copy Markdown
Contributor

Summary

What does this PR do, and why?

Closes #

Type of change

  • Bug fix
  • New feature
  • Refactor / chore
  • Docs

How was it tested?

Describe the verification (screens exercised, routes hit, seed data used). "It compiles"
is not enough — it should be verified in the running app.

Checklist (definition of done)

  • Verified end‑to‑end in the running app (backend :3001, frontend :3000)
  • UI is dark‑mode consistent and English (comments may stay Spanish)
  • Backend calls go through services/api.js (or a documented raw exception)
  • New routes tested and functional
  • Migrations generated and applied, no model ↔ migration drift
  • npm run lint passes (0 warnings)
  • No secrets committed

Screenshots (UI changes)

Before / after if relevant.

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