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

Implemented Add Drug Modal #21

Merged
merged 2 commits into from
Dec 4, 2024
Merged

Conversation

ValeriGuerrero
Copy link
Collaborator

Description

1) What does this PR do? 2) List out what you did.

1, It allows the user to add the drugs, and it has a modal button, I also connected the edit and delete buttons, but will wait to add those to the Medicine card.

Related Issue

_Provide a link to the related issue

Type of Changes

Feature update/change/addition, bug fix, styling update, etc.

Acceptance Criteria

Include the checked-off acceptance criteria from the issue.

Update Screenshots

Provide screenshots of the view before and after the changes.

Before

Screenshot 2024-11-30 at 6 13 18 PM

After

Screenshot 2024-11-30 at 6 45 46 PM

Testing Instructions

Just enter the information you want and click save, once the modal comes up it will have a message you can click the close button to close it. You'll be able to edit and update the medicine card, or delete it.

In a list format, tell us how to test this PR.

Learnings (Optional)

Document any things you learned or 'gotchas' you experienced.

I learned about modals and how they work, I also learned to not have the page scroll when the modal is open.

@ValeriGuerrero ValeriGuerrero linked an issue Dec 1, 2024 that may be closed by this pull request
Copy link
Collaborator

@GinaCastromonte GinaCastromonte left a comment

Choose a reason for hiding this comment

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

Looks great!

@sarross88
Copy link
Collaborator

Looks good to me too

@sarross88 sarross88 merged commit ab80cfc into main Dec 4, 2024
1 check passed
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.

Implement Add/Edit Drug Modal
3 participants