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

fixed skills sorting, day select, and cleaned up code #104

Merged
merged 1 commit into from
Jun 5, 2023

Conversation

GitLukeW
Copy link
Contributor

@GitLukeW GitLukeW commented Jun 5, 2023

Fixed the skills sorting feature, so now when a user clicks the drop down it will only show them skills that for mentors that have availability and not all skills that all mentors in the database know.

Fixed the date select. It was selecting the wrong day, so today was being past as yesterday. This is now fixed, so a user can schedule time with a mentor today.

Cleaned up unused code.

@netlify
Copy link

netlify bot commented Jun 5, 2023

Deploy Preview for momentum-mentors ready!

Name Link
🔨 Latest commit 250d600
🔍 Latest deploy log https://app.netlify.com/sites/momentum-mentors/deploys/647e74dbd102da0008f99261
😎 Deploy Preview https://deploy-preview-104--momentum-mentors.netlify.app/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@GitLukeW GitLukeW merged commit cc4a005 into main Jun 5, 2023
@GitLukeW GitLukeW deleted the MM-43-Skills-not-sorting branch June 5, 2023 23:59
capelhoworth added a commit to capelhoworth/Team_Production_System that referenced this pull request Aug 9, 2023
…for overlapping events. new bug discovered with ticket TeamProductionSystem#104 made to resolve it on backend
GitLukeW added a commit that referenced this pull request Sep 9, 2023
…rting

fixed skills sorting, day select, and cleaned up code
GitLukeW pushed a commit that referenced this pull request Sep 9, 2023
…ts. new bug discovered with ticket #104 made to resolve it on backend
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