Hospital Managment System Oracle APEX

[KhadijaAlloughai]

https://oracleapex.com/ords/r/alnoormanagmentsystem/alnoor-management-system/login?session=102399443962310

[github-actions]

Here are the Code Review Comments for the pull request titled "Hospital Management System Oracle APEX":

• Title Format: The title should begin with a ClickUp ticket ID followed by a colon, e.g., "86ex6r17g: Hospital Management System Oracle APEX".

• Description: The description of the pull request is empty. It should summarize the changes made in this PR.

• Indentation: Consistency in indentation is maintained throughout at 4 spaces.

• Blank Lines: There are a few unnecessary blank lines across the files:

  • DDL.sql: Line 3 is an unnecessary blank line; consider removing it.
  • DML.sql: Line 4 could be removed as well.

• Logical Errors:

  • In DDL.sql, the use of NULL within exception blocks could mask potential errors. It might be better to log or raise an appropriate message.

  • In DML.sql, the INSERT statements should verify patient_id and doctor_id exist before inserting appointments and admissions. This could prevent foreign key violations.

• Potential Improvements:

  • Consider adding ON DELETE CASCADE for relationships if you want related entries in dependent tables to be removed automatically when a record in the parent table is deleted.

  • In f104.sql, please check if there are any hardcoded values (like schema names) that may need modification depending on the environment.

All issues and suggestions are relevant to the changes made in this pull request.

opencode session  |  github run

[github-actions]

fatal: could not read Username for 'https://github.com': No such device or address

opencode session  |  github run