[No QA] Update Book-a-Hotel.md#79064
Conversation
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? |
|
@twisterdotcom 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] |
e33761f
into
stephanieelliott-patch-22
|
|
||
| # How to search and book a hotel | ||
| # How to book a hotel | ||
|
|
There was a problem hiding this comment.
Heading level violation: This should be ## (H2) instead of # (H1). Only the article title at the top should use H1. All major sections should use H2 (##).
| <div id="new-expensify" markdown="1"> | ||
|
|
||
| Book hotels easily using Expensify’s integrated travel tool. This guide covers how to search for hotels, filter options, complete your booking, and view your travel details in New Expensify. | ||
| Book hotels easily with Expensify Travel. This guide covers how to search for hotels, filter options, complete your booking, and view your travel details in New Expensify. |
There was a problem hiding this comment.
AI Readiness violation: Remove this introductory sentence. The article should start directly with the first H2 section. Introductory summaries should be in the YAML description field only (which is already present).
| To modify or cancel a booking: | ||
| 1. Tap the green ➕ **Create** button at the bottom of your screen, then choose **Book travel**. | ||
| 2. In the window that opens, click the **Trips** tab. | ||
| 3. Find your trip and click **Modify or Cancel** |
There was a problem hiding this comment.
AI Readiness violation: The answer "It will be SmartScanned and added to your next report automatically" uses the pronoun "It" without clear context. Start with the subject: "The hotel receipt will be SmartScanned and added to your next report automatically."
HelpDot Documentation ReviewOverall AssessmentThis PR updates the "Book a Hotel" documentation with improved clarity and consistency. The changes streamline navigation instructions, modernize terminology, and enhance the user experience. The updates are well-aligned with New Expensify's current interface patterns. Scores Summary
Key FindingsPositive Aspects:
Issues Identified:
RecommendationsPriority Actions:
Suggestions for Enhancement:
Files Reviewed
Overall Verdict: Strong documentation update that improves user experience. Addressing the heading hierarchy and terminology consistency issues would elevate this to excellent quality. |
| Book hotels easily with Expensify Travel. This guide covers how to search for hotels, filter options, complete your booking, and view your travel details in New Expensify. | ||
|
|
||
| # How to access the travel tool | ||
| ## Where to find Expensify Travel |
There was a problem hiding this comment.
AI Readiness improvement: Good change to H2! However, consider making the heading more specific by including "Expensify Travel" in the heading for better AI context. Suggestion: "## Where to find the Expensify Travel booking tool"
|
|
||
| From the left-hand menu, select **Reports > Trips**. | ||
| Click the green **+** button in the bottom-left corner, then choose **Book travel**. | ||
| Tap the green ➕ **Create** button at the bottom of your screen, then choose **Book travel**. |
There was a problem hiding this comment.
Readability violation: The instruction says "Tap the green ➕ Create button" but previously it was "Click the green + button". Ensure consistency - is this for mobile (tap) or desktop (click)? If it works on both, use "Click" as it's more universal, or specify the platform.
| Tap the green ➕ **Create** button at the bottom of your screen, then choose **Book travel**. | ||
|
|
||
| If you don’t see this option, reach out to your Account Manager or Concierge to schedule a travel demo and enable the feature for your account. | ||
| If you don’t see **Book travel**, ask a Workspace Admin to [enable Expensify Travel](https://help.expensify.com/articles/travel/company-setup/Enable-Travel-on-a-Workspace) on the workspace. |
There was a problem hiding this comment.
Terminology violation: Use "workspace" instead of "the workspace". Also consider capitalizing "Workspace Admin" for consistency with Expensify terminology: "ask a Workspace Admin to enable Expensify Travel on your workspace."
|
|
||
| # What happens next | ||
| ## What happens after booking with Expensify Travel | ||
|
|
There was a problem hiding this comment.
AI Readiness violation: The heading "What happens after booking with Expensify Travel" is verbose. Consider shortening while maintaining clarity: "## After booking" or "## What happens after booking"
| - You’ll receive a confirmation email after booking. | ||
| - The hotel receipt will automatically be added to your Expensify account. | ||
| - Note: If your hotel booking is post-paid, then you'll need to upload your final hotel bill to your Expensify account instead. | ||
| - If your hotel is post-paid, upload the final bill manually to your Expensify account after checkout. |
There was a problem hiding this comment.
Readability improvement: Good simplification! However, "upload the final bill manually" is slightly redundant - "upload" already implies manual action. Consider: "upload the final bill to your Expensify account after checkout."
|
|
||
| You can cancel or modify your booking if the hotel’s policy allows it. | ||
| You can cancel or modify the booking if the hotel’s policy allows. | ||
| Cancellation policies are shown during booking. Changes made via support incur a $25 booking change fee. |
There was a problem hiding this comment.
Readability violation: Remove "the" before "booking" - it reads more naturally as: "You can cancel or modify your booking if the hotel's policy allows."
|
|
||
| Manage bookings by going to **Reports > Trips > My Trips**. | ||
| To modify or cancel a booking: | ||
| 1. Tap the green ➕ **Create** button at the bottom of your screen, then choose **Book travel**. |
There was a problem hiding this comment.
Readability violation: Inconsistent terminology - earlier the instructions said "Tap" but here it says "Tap" again, and then "click" in step 3. Standardize to either "Click" throughout (for cross-platform) or specify when to use "Tap" vs "Click".
|
🚀 Deployed to staging by https://github.com/stephanieelliott in version: 9.3.5-0 🚀
|
|
@stephanieelliott, if we need to QA this, please add a repro steps |
|
🚀 Deployed to production by https://github.com/Beamanator in version: 9.3.5-7 🚀
|
Explanation of Change
Fixed Issues
$
PROPOSAL:
Tests
Offline tests
QA Steps
// TODO: These must be filled out, or the issue title must include "[No QA]."
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectioncanBeMissingparam foruseOnyxtoggleReportand notonIconClick)src/languages/*files and using the translation methodSTYLE.md) were followedAvatar, 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.ScrollViewcomponent to make it scrollable when more elements are added to the page.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
Android: mWeb Chrome
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari