Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding Functionality to Make an Appointment #3 #239

Merged
merged 7 commits into from
May 22, 2024
Merged

Adding Functionality to Make an Appointment #3 #239

merged 7 commits into from
May 22, 2024

Conversation

Srujitha-Raghava
Copy link
Contributor

#3

#Summary:
This pull request introduces a new functionality to the existing system, enabling users to create appointment forms.

#Changes Made:
1.Added new HTML file aptmnt.html to the project directory.
2.Implemented front-end design for the appointment form using HTML and CSS.

#Purpose:
The purpose of this pull request is to enhance the system's usability by providing users with a streamlined way to schedule appointments. By introducing this new feature, we aim to improve user experience and increase the efficiency of appointment scheduling within the system.

Screenshot 2024-05-20 142544

@Srujitha-Raghava
Copy link
Contributor Author

hey @Anishkagupta04 @varshith257 as said i have made some changes in the appointment form using hover effect and other colour changes.kindly,review it!

@varshith257
Copy link
Collaborator

@Srujitha-Raghava Are you from SVEC, Bhimavaram?

@Srujitha-Raghava
Copy link
Contributor Author

@varshith257 YES

@varshith257
Copy link
Collaborator

@varshith257 YES

Okay, I am from Narsapur :)

@varshith257 varshith257 linked an issue May 20, 2024 that may be closed by this pull request
@varshith257 varshith257 added enhancement New feature or request gssoc level2 medium (enhancements) labels May 20, 2024
@Srujitha-Raghava
Copy link
Contributor Author

@varshith257 YES

Okay, I am from Narsapur :)

That's good to know!

@varshith257
Copy link
Collaborator

@Srujitha-Raghava Incorporate above requested changes

@varshith257 varshith257 self-requested a review May 22, 2024 07:01
Copy link
Collaborator

@varshith257 varshith257 left a comment

Choose a reason for hiding this comment

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

Update the changes requested above

aptmnt.html Outdated
<label for="location">Your Location</label>
<select id="location" name="location" required>
<option value="" disabled selected>Your Location</option>
<option value="Location 1">Location 1</option>
Copy link
Collaborator

Choose a reason for hiding this comment

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

Better add some realistic locations in dropdown..Like Hyderabad, Delhi, Mumbai etc....

Copy link
Collaborator

Choose a reason for hiding this comment

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

@Srujitha-Raghava Please incorporate this changes and it's good to merge except this

appointment.html Outdated
Comment on lines 159 to 161
<option value="Location 1">Location 1</option>
<option value="Location 2">Location 2</option>
<option value="Location 3">Location 3</option>
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
<option value="Location 1">Location 1</option>
<option value="Location 2">Location 2</option>
<option value="Location 3">Location 3</option>
<option value="location 1">Delhi</option>
<option value="location 2">Bangalore</option>
<option value="location 3">Hyderabad</option>

Change them to this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@varshith257 sure,I will work on that

Copy link
Contributor Author

@Srujitha-Raghava Srujitha-Raghava left a comment

Choose a reason for hiding this comment

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

hey @varshith257 i have incorporated the requested changes,can you please check them!

Screenshot 2024-05-22 151922

@varshith257 varshith257 merged commit 02192af into Anishkagupta04:main May 22, 2024
1 check passed
@varshith257
Copy link
Collaborator

Thanks for the contribution

@Srujitha-Raghava Srujitha-Raghava deleted the aptmnt branch May 22, 2024 14:24
@Srujitha-Raghava Srujitha-Raghava restored the aptmnt branch May 22, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request gssoc level2 medium (enhancements)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants