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

feat: add enrollments/availabilities counter to event show page #204

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

ruilopesm
Copy link
Member

No description provided.

@netlify
Copy link

netlify bot commented Oct 14, 2023

Deploy Preview for coderdojobraga-app ready!

Name Link
🔨 Latest commit 088263c
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-app/deploys/652aceeea69f7b00080956c5
😎 Deploy Preview https://deploy-preview-204--coderdojobraga-app.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 configuration.

@reviewpad reviewpad bot added the app App related contributions label Oct 14, 2023
@reviewpad
Copy link

reviewpad bot commented Oct 14, 2023

AI-Generated Summary: This pull request introduces changes to the event page and Availability component in the application. It removes the buttonTitle property from the Availability component and consequently from the function calls in the event pages. Instead, it adds a counter to the title within both the Availability component and the event pages, which shows the number of mentors or enrolled ninjas depending on the context. In addition, it modifies the Availability function signature by removing the buttonTitle parameter. Overall, the pull request simplifies the code and enhances user interface by providing a count of mentors or enrolled ninjas.

@netlify
Copy link

netlify bot commented Oct 14, 2023

Deploy Preview for coderdojobraga-web canceled.

Name Link
🔨 Latest commit 088263c
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-web/deploys/652aceeec6e256000897e931

@netlify
Copy link

netlify bot commented Oct 14, 2023

Deploy Preview for coderdojobraga-maintenance canceled.

Name Link
🔨 Latest commit 088263c
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-maintenance/deploys/652aceee1f509900088ecf6f

@netlify
Copy link

netlify bot commented Oct 14, 2023

Deploy Preview for coderdojobraga-blog ready!

Name Link
🔨 Latest commit 088263c
🔍 Latest deploy log https://app.netlify.com/sites/coderdojobraga-blog/deploys/652aceeeea329b0008481b19
😎 Deploy Preview https://deploy-preview-204--coderdojobraga-blog.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 configuration.

@ruilopesm ruilopesm requested a review from danielsp45 October 14, 2023 17:58
@Afonso-santos Afonso-santos merged commit 6dae3cb into main Oct 16, 2023
15 checks passed
@Afonso-santos Afonso-santos deleted the rl/enrollments-counter branch October 16, 2023 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
app App related contributions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants