Skip to content

Fix compiling of marketing site after events page change#13500

Merged
parlough merged 1 commit into
mainfrom
fix/events-page-compile-failure
Jun 13, 2026
Merged

Fix compiling of marketing site after events page change#13500
parlough merged 1 commit into
mainfrom
fix/events-page-compile-failure

Conversation

@parlough

Copy link
Copy Markdown
Member

Fixes 8b1525f failing to run and deploy due to inconsistencies after applying the suggested changes through the GitHub UI.

@parlough parlough requested review from a team and abdallahshaban557 as code owners June 13, 2026 17:46

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request cleans up a duplicate declaration of the now variable, formats the cutoff variable assignment, and fixes a bug by replacing the undefined today variable with cutoff in the event filtering logic. There are no review comments, so I have no feedback to provide.

@abdallahshaban557 abdallahshaban557 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit 1022eac):

https://flutter-docs-prod--docs-pr13500-fix-events-page-compil-owkyc73s.web.app

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit 1022eac):

https://flutter-dev-230821--www-pr13500-fix-events-page-compil-2z2ws492.web.app

@parlough parlough merged commit 30ed47c into main Jun 13, 2026
14 checks passed
@parlough parlough deleted the fix/events-page-compile-failure branch June 13, 2026 18:05
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.

3 participants