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.

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

Remove Degree and Semester fields from application form #2

Merged
merged 1 commit into from
Jan 6, 2025

Conversation

GoliathLabs
Copy link
Owner

@GoliathLabs GoliathLabs commented Jan 6, 2025

Remove the "Degree" and "Semester" fields from the application form UI and update the backend to stop expecting or storing these fields.

  • Backend Changes:

    • Remove the "degree" and "semester" columns from the "applications" table in backend/src/db/migrations/0000_empty_texas_twister.sql.
    • Remove the "degree" and "semester" columns from the "applications" table definition in backend/src/db/migrations/meta/0000_snapshot.json.
    • Remove the "degree" and "semester" fields from the applications table definition in backend/src/db/schema.ts.
    • Remove the "degree" and "semester" fields from the InsertApplication and UpdateApplication schemas in backend/src/models/application.ts.
  • Frontend Changes:

    • Remove the input fields for "Degree" and "Semester" in frontend/app/components/ApplicationDetailDialog.vue.
    • Remove the columns for "Degree" and "Semester" in the applications table in frontend/app/pages/index.vue.
    • Remove the "degree" and "semester" fields from the AddApplication schema in frontend/app/types/applicaion.schema.ts.

For more details, open the Copilot Workspace session.

Remove the "Degree" and "Semester" fields from the application form UI and update the backend to stop expecting or storing these fields.

* **Backend Changes:**
  - Remove the "degree" and "semester" columns from the "applications" table in `backend/src/db/migrations/0000_empty_texas_twister.sql`.
  - Remove the "degree" and "semester" columns from the "applications" table definition in `backend/src/db/migrations/meta/0000_snapshot.json`.
  - Remove the "degree" and "semester" fields from the `applications` table definition in `backend/src/db/schema.ts`.
  - Remove the "degree" and "semester" fields from the `InsertApplication` and `UpdateApplication` schemas in `backend/src/models/application.ts`.

* **Frontend Changes:**
  - Remove the input fields for "Degree" and "Semester" in `frontend/app/components/ApplicationDetailDialog.vue`.
  - Remove the columns for "Degree" and "Semester" in the applications table in `frontend/app/pages/index.vue`.
  - Remove the "degree" and "semester" fields from the `AddApplication` schema in `frontend/app/types/applicaion.schema.ts`.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/GoliathLabs/applica?shareId=XXXX-XXXX-XXXX-XXXX).
@GoliathLabs GoliathLabs merged commit 3b61cba into master Jan 6, 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.

1 participant