[No QA] Remove rental cars from Consolidated Travel Billing help doc - #96765
Conversation
|
A preview of your ExpensifyHelp changes have been deployed to https://9ac2880c.helpdot.pages.dev ⚡️ Updated articles: |
HelpDot Documentation ReviewOverall AssessmentThis is a small, focused content correction to the Book Travel Using Consolidated Travel Billing article. It removes rental car / car booking references from two locations: the bookable travel types list (step 4) and the post-booking receipt guidance. The change is factually motivated (car bookings are not supported by CTB) and improves the accuracy of the doc. Only two lines are modified, so this review is scoped strictly to those edits. Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: Detailed line-by-line feedback has been provided as inline comments. |
|
🚀 Deployed to staging by https://github.com/blimpich in version: 9.4.43-0 🚀
|
|
🚀 Deployed to production by https://github.com/yuwenmemon in version: 9.4.43-1 🚀
Bundle Size Analysis (Sentry): |
Explanation of Change
The Book Travel Using Consolidated Travel Billing help doc told employees they could book rental cars using Consolidated Travel Billing. That's incorrect — car bookings aren't supported by CTB. This removes rental cars from the list of bookable travel types and from the post-booking receipt guidance.
Fixed Issues
$ https://expensify.slack.com/archives/C05S5EV2JTX/p1784690321172779?thread_ts=1784667244.731329&cid=C05S5EV2JTX
PROPOSAL:
Tests
Offline tests
QA Steps
See [No QA] in title.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectionAvatar, I verified the components usingAvatarare working as expected)StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))npm run compress-svg)Avataris modified, I verified thatAvataris working as expected in all cases)Designlabel and/or tagged@Expensify/designso the design team can review the changes.mainbranch was merged into this PR after a review, I tested again and verified the outcome was still expected according to theTeststeps.Screenshots/Videos
Android: Native
N/A - docs-only content change.
Android: mWeb Chrome
N/A - docs-only content change.
iOS: Native
N/A - docs-only content change.
iOS: mWeb Safari
N/A - docs-only content change.
MacOS: Chrome / Safari
N/A - docs-only content change.