Skip to content

Conversation

divyanshkhetan
Copy link

Issue that this pull request solves

Issue - Book Appointment feature does not work on backend.

Proposed changes

  1. Check if the backend has been developed for booking the appointment or not.
  2. If not, create the backend for this functionality from scratch.
  3. If yes, find and fix if any bugs are present.

Brief description of what is fixed or changed

  1. The backend section for the feature was found missing and has been added on the backend.
  2. A few changes have been made to frontend to generate appropriate error messages for user-friendliness.

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

Copy link

@NuthanReddy646 NuthanReddy646 left a comment

Choose a reason for hiding this comment

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

Approve @akshatnema

@akshatnema akshatnema changed the base branch from os-2 to test March 28, 2023 06:20
Comment on lines 69 to 91
function trimfield(str)
{
return str.replace(/^\s+|\s+$/g,'');
}





$('#appo').on('click',function(e){
e.preventDefault();

var tod = new Date(new Date().setHours(0, 0, 0, 0));
var today = new Date(new Date());
var now = today.getHours();

if(trimfield(document.getElementById("description").value) == '')
{
$('#desc').html('*Please fill out the Description!');
document.getElementById("description").focus();
}


Copy link
Collaborator

Choose a reason for hiding this comment

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

Please remove unnecessary whitespaces in the code.

Copy link
Author

Choose a reason for hiding this comment

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

@akshatnema done. PTAL.

@akshatnema akshatnema merged commit d4d4a00 into FusionIIIT:test Apr 15, 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.

3 participants