Skip to content

Conversation

msrajawat298
Copy link
Contributor

Added

  • Added .DS_Store to the .gitignore file to prevent it from being tracked.
  • Added npm run lint, npm run test:coverage to the .husky/pre-commit file to lint, and run test coverage before committing.
  • Added .sequelizerc file to configure paths for Sequelize.
  • Added Sequelize ORM link to the README file.
  • Added config/auth.config.js file for authentication configurations.
  • Added engines field to package.json specifying Node.js version requirement.
  • Added database/custom_migration/sync-db.js file to synchronize the database.
  • Added database/seeders/20240412065335-seed-roles.cjs file for seeding roles.

Changed

  • Updated app/controllers/auth.controller.js and app/middleware/authJwt.js to import config/auth.config.js from the correct path.
  • Changed references to config/db.config.js to use ES6 destructuring in app/models/index.js.
  • Moved config/auth.config.js to the config directory.
  • Deleted config/config.json as it's no longer needed.
  • Moved app/config/db.config.js to the config directory.
  • Removed unused migration files: migrations/20240412074035-create-roles.cjs, migrations/20240412074035-create-users.cjs, migrations/20240412074305-create-user-roles.cjs.
  • Removed unused model files: models/index.cjs.
  • Updated package.json scripts to use correct paths for migrations and seed generation.

Fixed

  • Fixed incorrect path in custom_migration/sync-db.js.
  • Fixed incorrect path in seeders/20240412065335-seed-roles.cjs.
  • Fixed package.json scripts to properly specify migration and seed generation names.

@msrajawat298 msrajawat298 merged commit 486dfb1 into main Apr 22, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

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