[No QA] Docs updates for E/A#95316 Note Device management list sort order#95411
Conversation
Co-authored-by: cretadn22 <cretadn22@users.noreply.github.com>
|
A preview of your ExpensifyHelp changes have been deployed to https://8ec935ec.helpdot.pages.dev ⚡️ Updated articles: |
|
@MelvinBot Ready for review |
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? |
|
🤖 Done — this PR is now marked Ready for review. One note: the Tests and QA sections in the PR body still contain the |
|
@cretadn22 @lydiabarclay One of you needs to 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] |
Reviewer Checklist
Screenshots/VideosAndroid: HybridAppAndroid: mWeb ChromeiOS: HybridAppiOS: mWeb SafariMacOS: Chrome / Safari |
HelpDot Documentation ReviewOverall AssessmentThis PR makes a minimal, well-scoped follow-up documentation change to Manage-Logged-in-Devices.md, adding a single explanatory sentence noting that the Device management list is ordered by most recent activity. This aligns the article with the new sort behavior introduced in #95316. The change is surgical and does not alter structure, headings, or metadata. Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: This is a top-level summary review of the proposed diff only. |
|
🚀 Deployed to staging by https://github.com/arosiclair in version: 9.4.33-0 🚀
|
|
🚀 Deployed to production by https://github.com/lakchote in version: 9.4.33-8 🚀
Bundle Size Analysis (Sentry): |
Explanation of Change
Follow-up help site update for #95316, which sorts the Device management list so the most recently used or added device appears first.
The article
Manage-Logged-in-Devices.mddescribed the device list but did not mention its ordering. This adds one sentence noting that the list is ordered by most recent activity, matching the new behavior. No other help site articles reference the device list order.Fixed Issues
$ #95307
Tests
// TODO: The human co-author must fill out the tests you ran before marking this PR as "ready for review"
// Please describe what tests you performed that validates your changed worked.
QA
// TODO: The human co-author must fill out the QA tests you ran before marking this PR as "ready for review".
// Please describe what QA needs to do to validate your changes and what areas do they need to test for regressions.
@cretadn22, please review the files changed and confirm they reflect the current behavior. Then mark this PR
Ready for review.