Skip to content

Update CubePilot branding and streamline scan UX#77

Merged
markcoleman merged 2 commits intomainfrom
iterate-ux
Feb 27, 2026
Merged

Update CubePilot branding and streamline scan UX#77
markcoleman merged 2 commits intomainfrom
iterate-ux

Conversation

@markcoleman
Copy link
Copy Markdown
Owner

Summary

  • rename the app to Cube Pilot across assets, launch screens, and strings
  • refresh the icon, launch image, and UI text to match the new naming
  • revamp the scanner flow layout and guidance text for a simpler, more balanced experience while keeping the workflow cues
  • harden the camera session handling to avoid re-entry errors

Testing

  • Not run (not requested)

@github-actions
Copy link
Copy Markdown
Contributor

⚠️ This PR is very large. Consider breaking it into smaller PRs for easier review.

PR Stats:

  • Lines added: 714
  • Lines deleted: 149
  • Files changed: 23

Tips for large PRs:

  • Break into smaller, focused PRs
  • Ensure comprehensive test coverage
  • Add detailed descriptions for each change

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9e579a7197

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +302 to +303
INFOPLIST_KEY_UILaunchScreen_Generation = NO;
INFOPLIST_KEY_UILaunchStoryboardName = LaunchScreen;
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Bundle the launch storyboard before enabling it

This change disables the generated launch screen and sets UILaunchStoryboardName to LaunchScreen, but the project file still only lists Assets.xcassets in PBXResourcesBuildPhase and has no PBXFileReference/PBXBuildFile entry for LaunchScreen.storyboard, so iOS builds from this commit won’t include a compiled LaunchScreen.storyboardc. On app startup, the system tries to load that storyboard and can fail to present the launch screen (and in practice may terminate at launch for missing launch storyboard resources).

Useful? React with 👍 / 👎.

@markcoleman markcoleman merged commit c11c421 into main Feb 27, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant