Skip to content

Add Al Noor Hospital Management SQL scripts. - #609

Open
Alharith99 wants to merge 1 commit into
mainfrom
feature/alnoor-hospital-sql
Open

Add Al Noor Hospital Management SQL scripts.#609
Alharith99 wants to merge 1 commit into
mainfrom
feature/alnoor-hospital-sql

Conversation

@Alharith99

Copy link
Copy Markdown
Collaborator

Schema, views, triggers, sample data, and install helpers for the APEX HMS assignment.

Schema, views, triggers, sample data, and install helpers for the APEX HMS assignment.
@github-actions

Copy link
Copy Markdown

Code Review Comments

  1. Pull Request Title:

    • ❌ Title does not start with a ClickUp ticket ID. Please update to follow the format: "clickup_id: title".
  2. Pull Request Description:

    • ❌ The description should specifically reflect the changes made (e.g., mention key functionalities like added tables, triggers etc.).
  3. Indentation:

    • ✅ The code consistently uses 4 spaces for indentation.
  4. Blank Lines:

    • ❌ Some SQL files contain unnecessary blank lines, which should be removed to follow clean coding practices.
  5. Logic and Improvement:

    • ❌ No obvious logical errors, but consider implementing error handling in triggers (e.g., check for NULL values where applicable).

After reviewing the changes, please address the issues highlighted above.

New%20session%20-%202026-07-21T12%3A52%3A44.493Z
opencode session  |  github run

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.

5 participants