Skip to content

Conversation

divyanshkhetan
Copy link

Proposed changes

  • Fixed the code errors in the template file.
  • Increased the char limit for reason field in Ambulance_request to 500 so as to provide enough space for appropriate description of reason.
  • Added migration changes to resolve previous merge conflicts and add the new change to the model.

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
image

@NuthanReddy646
Copy link

Approve @akshatnema

var end=new Date(endDate);
var today = new Date(new Date().setHours(0, 0, 0, 0));

console.log("Was here:::")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
console.log("Was here:::")

});
</script>
<p id="dte"></p>
<!-- <p id="dte"></p> -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<!-- <p id="dte"></p> -->

});
</script>
<p id="dte"></p>
<!-- <p id="dte"></p> -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<!-- <p id="dte"></p> -->

<div class="ui fluid input">
<textarea class="ui textarea" rows="2" id="reason" name="reason"></textarea>
<p id="reas"></p>
<!-- <p id="reas"></p> -->
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
<!-- <p id="reas"></p> -->

Copy link
Author

Choose a reason for hiding this comment

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

@akshatnema I've applied the suggested changes. PTAL.

@akshatnema
Copy link
Collaborator

Review required from @PranshuNayak and @samay-rgb

@divyanshkhetan divyanshkhetan changed the title Week 12: OS-2: Fixed all the ambulance use cases OS-2: Week-10: Fixed all the ambulance use cases Apr 15, 2023
PranshuNayak
PranshuNayak previously approved these changes Apr 16, 2023
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.

LGTM!

Copy link
Collaborator

Choose a reason for hiding this comment

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

Do not push indentation changes

Copy link
Collaborator

Choose a reason for hiding this comment

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

Remove this file from PR.

Copy link
Author

Choose a reason for hiding this comment

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

@PranshuNayak PranshuNayak dismissed their stale review April 16, 2023 05:49

Remove the pushed migrations from schloarship module.

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.

LGTM!

@akshatnema akshatnema merged commit 72863a8 into FusionIIIT:test Apr 16, 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