-
Notifications
You must be signed in to change notification settings - Fork 3
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
1431: Add app build config app #1654
Conversation
Updates to support xcode 16
…lfabrik/entitlementcard into 1431-add-app-build-config-koblenz
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tested on android, works as expected. What do you mean by "check that deeplink files are created"?
export { default as buildConfigBayern } from "./bayern" | ||
export { default as buildConfigKoblenz } from "./koblenz" | ||
export type { DeeplLinkingConfig, BuildConfigType } from "./types" | ||
export * from "./constants" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🔧 I think your formatting is off (double vs single quotes).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same for other files. Shouldn't this be caught by the CI pipeline/prettier?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is quite inconsistent currently, as these ts files are unter /frontend and prettier is only in /administration. See e.g. build-config/bayern/index has also double quotes. I can adjust this, but i would prefer to do it in a separate PR, so this PR does not get too big.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh okay, thanks. Yes, then a separate PR makes sense.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very good work 👍
Tested on ios and android.
Would be nice if you can adjust this one line i mentioned in my comment before
Then its good to go
Please add issues for the two minor things i mentioned there
When building administration in the build folder there are the files apple-app-site-association and assentlinks.json for each project |
They are already in the issue in the description of the PR, i added the console output for the missing english translations |
okay i didn't find adjust theme color, so i added this to #1653 |
Short description
Added app build config for koblenz
Proposed changes
Still open tasks due to currently still missing data will be done in #1653
Known issue, will be fixed in separate task: #1686
Testing
Start the app with koblenz flavour
check if deep link files are created properly
check on android and ios
check if correct intro screens and images / icons are displayed
check if correct publisher text is displayed
check if map is centered correctly to koblenz
Resolved issues
Fixes: #1431