Skip to content

Conversation

divyanshkhetan
Copy link

@divyanshkhetan divyanshkhetan commented Apr 16, 2023

Proposed changes

  • Fixed stock system
  • Fixed doctor's model
  • Added Hospital Add view

Types of changes

Put an x in the boxes that apply

  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Other (please describe):

Checklist

Put an x in the boxes that apply

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have created new branch for this pull request
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • My changes does not break the current system and it passes all the current test cases.

Screenshots

image

@divyanshkhetan divyanshkhetan changed the title Medicine OS-2: Week 11: Fixed Hospital System and Medicine System Apr 16, 2023
@divyanshkhetan
Copy link
Author

@akshatnema please look into this.

@NuthanReddy646
Copy link

Approve @akshatnema

Copy link
Collaborator

@akshatnema akshatnema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review required from @PranshuNayak and @samay-rgb

Copy link
Collaborator

@PranshuNayak PranshuNayak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please make sure to do all necessary changes at once while changing the model schema. In the previous PR, you added contact_no now you are removing it. If possible try to give all changes in a single migration file.

@akshatnema akshatnema merged commit c5a3ff7 into FusionIIIT:test Apr 17, 2023
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