Skip to content

Added 2 images and 2 new consultants - #13866

Merged
abdallahshaban557 merged 2 commits into
mainfrom
consultants-sep8
Sep 9, 2026
Merged

Added 2 images and 2 new consultants#13866
abdallahshaban557 merged 2 commits into
mainfrom
consultants-sep8

Conversation

@htrolle-epam

@htrolle-epam htrolle-epam commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Description of what this PR is changing or adding, and why:

Issues fixed by this PR (if any):

PRs or commits this PR depends on (if any):

Presubmit checklist

  • If you are unwilling, or unable, to sign the CLA, even for a tiny, one-word PR, please file an issue instead of a PR.
  • If this PR is not meant to land until a future stable release, mark it as draft with an explanation.
  • This PR follows the Google Developer Documentation Style Guidelines—for example, it doesn't use i.e. or e.g., and it avoids I and we (first-person pronouns).
  • This PR uses semantic line breaks
    of 80 characters or fewer.

@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 two new consultant entries, "Tapped GmbH" and "Here2Build Apps GmbH", to the partners data file. The review feedback correctly identifies that both entries are missing the required options_GCP_partner field, which would cause a runtime parsing error due to it being a non-nullable field in the data model. A code suggestion was provided to resolve this issue.

Comment thread sites/www/content/consultants/data/partners.yaml Outdated
@htrolle-epam

Copy link
Copy Markdown
Contributor Author

Hello @abdallahshaban557 whenever you have an opportunity, please help to review and approve it is procceds

@flutter-website-bot

flutter-website-bot commented Sep 9, 2026

Copy link
Copy Markdown
Collaborator

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

https://flutter-docs-prod--docs-pr13866-consultants-sep8-a0bgd0ea.web.app

@flutter-website-bot

Copy link
Copy Markdown
Collaborator

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

https://flutter-dev-230821--www-pr13866-consultants-sep8-y70gq5gf.web.app

@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

@abdallahshaban557
abdallahshaban557 merged commit 2937512 into main Sep 9, 2026
13 checks passed
@abdallahshaban557
abdallahshaban557 deleted the consultants-sep8 branch September 9, 2026 16:49
sfshaza2 pushed a commit that referenced this pull request Sep 9, 2026
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