Skip to content

Test ad-hoc build cache for #97110 - #97672

Open
mjasikowski wants to merge 4 commits into
mainfrom
test-adhoc-build-97110
Open

Test ad-hoc build cache for #97110#97672
mjasikowski wants to merge 4 commits into
mainfrom
test-adhoc-build-97110

Conversation

@mjasikowski

Copy link
Copy Markdown
Contributor

Purpose

Testing PR for #97110 — Web ad-hoc builds serve stale code (v2 shows v1's changes).

Trivial, visible UI change (success/green color swapped to blue) so a build change is easy to eyeball. Plan: trigger an ad-hoc build, push another commit changing the color again, trigger a second ad-hoc build, and confirm it reflects the new commit instead of serving the previous build's output.

Not intended to merge.

Test plan

  • Trigger ad-hoc build (v1), confirm blue success color shows
  • Push follow-up commit changing color again
  • Trigger ad-hoc build (v2), confirm it shows the new color, not v1's

rinej and others added 2 commits July 28, 2026 13:25
Trivial, visible UI change to verify ad-hoc builds pick up new
commits. Related to #97110.
@mjasikowski
mjasikowski requested review from a team as code owners August 3, 2026 09:54
@melvin-bot
melvin-bot Bot requested review from JS00001 and joekaufmanexpensify and removed request for a team August 3, 2026 09:54
@melvin-bot

melvin-bot Bot commented Aug 3, 2026

Copy link
Copy Markdown

@JS00001 Please copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

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

Copy link
Copy Markdown

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: a52d86d1bc

ℹ️ About Codex in GitHub

Codex has been enabled to automatically 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 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread src/styles/theme/themes/light.ts Outdated
dangerPressed: colors.redHover,
warning: colors.yellow400,
success: colors.green400,
success: colors.blue400,

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 Revert the test-only success color swap

If this commit is merged, light-theme users get the global theme.success token as blue while the rest of the success palette still stays green (successHover, successPressed, dark theme, and components like styles.buttonSuccess/badgeSuccessStrong all assume this semantic is green). Since this token is used across success buttons, status indicators, selected states, and Onfido theming, the ad-hoc-build probe would ship a broad production UI regression rather than a test-only marker.

Useful? React with 👍 / 👎.

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🚧 mjasikowski has triggered a test Expensify/App build. You can view the workflow run here.

@github-actions

This comment has been minimized.

@codecov

codecov Bot commented Aug 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ Changes either increased or maintained existing code coverage, great job!

Files with missing lines Coverage Δ
src/App.tsx 100.00% <ø> (ø)
src/CONST/index.ts 94.81% <ø> (ø)
src/DeepLinkHandler.tsx 86.53% <ø> (ø)
src/ONYXKEYS.ts 100.00% <ø> (ø)
src/SCREENS.ts 100.00% <ø> (ø)
src/components/AccountManagerBookCallButton.tsx 100.00% <ø> (ø)
src/components/AddPaymentCard/PaymentCardForm.tsx 90.32% <ø> (+0.84%) ⬆️
src/components/AddressForm.tsx 92.64% <100.00%> (+0.10%) ⬆️
src/components/AvatarButtonWithIcon.tsx 100.00% <100.00%> (ø)
src/components/AvatarSelector.tsx 100.00% <100.00%> (ø)
... and 172 more
... and 565 files with indirect coverage changes

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🚧 mjasikowski has triggered a test Expensify/App build. You can view the workflow run here.

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🧪🧪 Use the links below to test this adhoc build on Android, iOS, and Web. Happy testing! 🧪🧪
Built from App PR #97672.

Android 🤖 iOS 🍎
⏩ SKIPPED ⏩ ⏩ SKIPPED ⏩
The build for Android was skipped The build for iOS was skipped
Web 🕸️
https://97672.pr-testing.expensify.com
Web

👀 View the workflow run that generated this build 👀

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