Skip to content

Create holafly showcase page - #13876

Draft
abdallahshaban557 wants to merge 6 commits into
flutter:mainfrom
abdallahshaban557:create_holafly_video_showcase
Draft

Create holafly showcase page#13876
abdallahshaban557 wants to merge 6 commits into
flutter:mainfrom
abdallahshaban557:create_holafly_video_showcase

Conversation

@abdallahshaban557

Copy link
Copy Markdown
Contributor

Adds the official Holafly customer showcase to flutter.dev/showcase/holafly.

  • Adds case study markdown content in sites/www/content/showcase/holafly.md detailing Holafly's rewrite in Flutter, architecture with Pigeon, and AI-driven localization with Gemini.
  • Adds Holafly case study images and logo in sites/www/content/showcase/images/third_party/case_studies/holafly/.
  • Adds Holafly logo to sites/www/content/images/third_party/logos/holafly.png and replaces Dream11 with Holafly in the homepage community logo garden.
  • Adds Holafly entry to sites/www/content/showcase/data/companies.yaml.
  • Updates showcase authoring skill instructions and removes unused card generation script.

@abdallahshaban557
abdallahshaban557 requested a review from a team as a code owner September 10, 2026 20:25
@abdallahshaban557
abdallahshaban557 marked this pull request as draft September 10, 2026 20:25

@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 adds a new showcase case study for Holafly, updating the homepage logo grid, adding Holafly to the companies list, and updating the showcase generation guidelines (SKILL.md and blog-format.md) to require descriptive, story-specific section headers. It also removes the automatic showcase card generation script. Feedback on the pull request points out two issues in the new holafly.md file: the card frontmatter field should point to the generated card image instead of the logo, and the successMetrics list should use the schema-required perc key instead of text.

Comment thread sites/www/content/showcase/holafly.md Outdated
Comment thread sites/www/content/showcase/holafly.md
@flutter-website-bot

flutter-website-bot commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Staged preview of the updated docs.flutter.dev site (updated for commit 2425f24):

https://flutter-docs-prod--docs-pr13876-create-holafly-video-s-zwpycx89.web.app

@flutter-website-bot

flutter-website-bot commented Sep 10, 2026

Copy link
Copy Markdown
Collaborator

Staged preview of the updated flutter.dev site (updated for commit 2425f24):

https://flutter-dev-230821--www-pr13876-create-holafly-video-s-b1lmhpid.web.app

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.

2 participants