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

Mentor dashboard buttons #849

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

Uthra-karunakaran
Copy link
Contributor

Related Issue

i just set up the buttons for calendar but there is an issues with setting it up for the queries , because of the " isTicketFormVisible" also the ticketform is not closing like the other components unless we close it ourself .

Description of Changes

  • changed the buttons used in the "./components/mentorDashboard/calendar.js" to use our component buttons primary and secondary
  • added logic to switch styles

Checklist:

  • My code adheres to the established style guidelines of this project.
  • I have conducted a self-review of my code.
  • I have included comments in areas that may be difficult to understand.
  • I have made corresponding updates to the project documentation.
  • My changes have not introduced any new warnings.

Screenshots

Original Updated
Original screenshot Updated screenshot

Please provide any necessary screenshots to illustrate the changes made.

Copy link

vercel bot commented Sep 12, 2024

@code-par-learn is attempting to deploy a commit to the anmode's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

Thank you, @code-par-learn, for creating this pull request and contributing to GitHub-ReadMe!
Our review team will thoroughly review the pull request and will reach out to you soon!
Please make sure you have marked all the completed tasks as done.
We appreciate your patience and contribution!

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.

1 participant