Skip to content

Conversation

@aayushi2882
Copy link

@aayushi2882 aayushi2882 commented Dec 18, 2025

This PR improves the empty state of the “Your upcoming events” section
by displaying a clear informational message when no upcoming events exist.

No changes were made to CTA buttons or event navigation logic.

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Sorry @aayushi2882, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@mariobehling mariobehling requested a review from Copilot December 18, 2025 22:17
@mariobehling
Copy link
Member

Which issue is this related to? Please also add screenshots.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR enhances the user experience of the dashboard by adding a clear informational message when there are no upcoming events. The implementation adds an {% else %} clause to display "You don't have any upcoming events yet." when the upcoming list is empty.

  • Adds an empty state message for the upcoming events section using an alert-info component
  • Restructures the template to include conditional rendering based on the presence of upcoming events
  • Maintains the existing "Create a new event" button and event list functionality

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 70 to 71
{% if upcoming %}
<p class="">
Copy link

Copilot AI Dec 18, 2025

Choose a reason for hiding this comment

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

This creates duplicate logic for checking the upcoming variable. Lines 51-62 check if upcoming to display events, and lines 70-76 also check if upcoming to show the "View all future events" link. This check should be consolidated to avoid redundant conditional logic.

Copilot uses AI. Check for mistakes.
@aayushi2882
Copy link
Author

aayushi2882 commented Dec 19, 2025

Thanks for the review!

This PR is related to issue #1506.

The change improves the empty state of the “Your upcoming events” section by showing a clear informational message when no upcoming events are available, instead of leaving the section blank.

The update is limited to a small conditional change in the dashboard template and does not affect any backend logic.

Running the full Eventyay stack locally is quite heavy, so I focused on a minimal and safe template-level change that is easy to review directly from the diff.

Please let me know if you’d like me to add anything else.

updated code snippet:
{% if upcoming %}
{% for w in upcoming %}
...
{% endfor %}
{% else %}

{% trans "You don’t have any upcoming events yet." %}
{% endif %}

Copy link
Member

@mariobehling mariobehling left a comment

Choose a reason for hiding this comment

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

Please provide screenshots.

Copy link
Member

@mariobehling mariobehling left a comment

Choose a reason for hiding this comment

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

No screenshots were provided so far. Please provide screenshots.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants