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

[Deep link] Generate AASA template when it doesn't exist #8345

Merged
merged 2 commits into from
Sep 27, 2024

Conversation

hannah-hyj
Copy link
Member

@hannah-hyj hannah-hyj commented Sep 24, 2024

We currently don't have an API for AASA generation. However, for users that don't have an AASA file, we can provide a template that's easy to copy-paste.

Pre-condition - Developer has an associated domain in "entitlements", but EXISTENCE check failed at ERROR severity level.

Assumption - The developer has appID as TEAMID.bundle.id and a associated domain like www.example.com

Generated template (Apple documentation):

Pre-launch Checklist

  • I read the Contributor Guide and followed the process outlined there for submitting PRs.
  • I read the Tree Hygiene wiki page, which explains my responsibilities.
  • I read the Flutter Style Guide recently, and have followed its advice.
  • I signed the CLA.
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or there is a reason for not adding tests.

build.yaml badge

If you need help, consider asking for help on Discord.

@hannah-hyj hannah-hyj requested a review from a team as a code owner September 24, 2024 22:13
@hannah-hyj hannah-hyj requested review from kenzieschmoll and removed request for a team September 24, 2024 22:13
Copy link
Member

@kenzieschmoll kenzieschmoll left a comment

Choose a reason for hiding this comment

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

a couple comments but lgtm

_FailureDetails(errors: [error]),
],
),
error == IosDomainError.existence
Copy link
Member

Choose a reason for hiding this comment

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

the ternary check can be when defining the children of _IssuesBorderWrap since that is repeated:

_IssuesBorderWrap(
  children: error == IosDomainError.existence ? [...] : [...],
)

_FailureDetails(
errors: [error],
oneFixGuideForAll:
'To fix this issues, add an Apple-App-Site-Association file at the following location: '
Copy link
Member

Choose a reason for hiding this comment

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

r/issues/issue

@hannah-hyj hannah-hyj merged commit f1117d3 into flutter:master Sep 27, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants