Skip to content

Add preview flavor - #3415

Draft
julien4215 wants to merge 1 commit into
mainfrom
firebase-preview
Draft

Add preview flavor#3415
julien4215 wants to merge 1 commit into
mainfrom
firebase-preview

Conversation

@julien4215

Copy link
Copy Markdown
Collaborator

Reopening of #3396

@julien4215

julien4215 commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator Author

@julien4215 sorry I had to revert because this broke iOS. Can you reopen it please?

I'll have to make it work on iOS before merging, but can't do it now. Leave it as a draft for now, so we know it is not ready.

I didn't realize it would break iOS. I assume it is because there is a default flavor set in the pubspec.yaml that must be used by each platform. This is annoying because we either need to also add flavors to iOS or set no default flavor but then the usual flutter run command will have to become flutter run --flavor prod on android.

Related Flutter issue flutter/flutter#176820

@veloce
veloce marked this pull request as draft July 8, 2026 14:26
@julien4215
julien4215 force-pushed the firebase-preview branch 2 times, most recently from 86c63f6 to d947a88 Compare July 16, 2026 12:04
@julien4215

Copy link
Copy Markdown
Collaborator Author

@veloce I think renaming the Xcode scheme to prod should solve the build issue on iOS. If that is preferable, we can rename the prod flavor to Runner instead.

Or we can remove the default-flavor from the pubspec.yaml but each flutter command that builds the app on android will need to specify a flavor, e.g. flutter run --flavor prod.

What do you think would be the best approach?

@veloce

veloce commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

@veloce I think renaming the Xcode scheme to prod should solve the build issue on iOS. If that is preferable, we can rename the prod flavor to Runner instead.

Or we can remove the default-flavor from the pubspec.yaml but each flutter command that builds the app on android will need to specify a flavor, e.g. flutter run --flavor prod.

What do you think would be the best approach?

Not sure yet, I have to investigate. Ideally android and iOS should behave the same way so the work you did for android needs to be done for iOS too.

@veloce
veloce force-pushed the firebase-preview branch from 3ad7f62 to c6ee7c8 Compare July 26, 2026 14:50
@veloce

veloce commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

I rebased the branch fyi @julien4215 ; will add support for iOS from here.

preview.test.example.com and a separate FCM configuration for the
preview website.
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