Skip to content

Fix/es6 controllers#63

Merged
agaesh merged 12 commits intofix/es6-migrationfrom
fix/es6-controllers
Oct 11, 2025
Merged

Fix/es6 controllers#63
agaesh merged 12 commits intofix/es6-migrationfrom
fix/es6-controllers

Conversation

@agaesh
Copy link
Copy Markdown
Owner

@agaesh agaesh commented Oct 11, 2025

Here’s a solid and professional pull request comment you can post for that set of commits 👇


🧱 Pull Request: Refactor Controllers to ES Module Syntax

Overview

This PR completes the migration of all controller files from CommonJS to ES Module (ESM) syntax.
The goal is to ensure full compatibility with modern Node.js environments and align with the ongoing ES6 refactor initiative across the codebase.


Controllers Updated

  • DepartmentController — migrated and optimized CRUD functions
  • InstructorController — cleaned and reorganized create/update logic
  • ProgramCourseController — standardized imports and refactored handlers
  • UserController — improved function structure and cookie management

Improvements

  • Unified syntax using import / export
  • Simplified function structures for better readability
  • Enhanced code consistency across all controllers
  • Prepared controllers for upcoming ESM-based service integrations

Next Steps

  • Refactor corresponding service and route layers

Summary:
All controllers are now fully ESM-compliant and ready for integration testing.
This refactor modernizes the project structure and reduces potential module resolution conflicts.

@agaesh agaesh linked an issue Oct 11, 2025 that may be closed by this pull request
7 tasks
@agaesh agaesh merged commit c08a07b into fix/es6-migration Oct 11, 2025
@agaesh agaesh self-assigned this Oct 11, 2025
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.

Title: Convert all controllers to ES6 module syntax

1 participant