[No QA] Move docs to new Rillet hub on help site - #99454
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? |
|
@mjasikowski @jliexpensify 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] |
There was a problem hiding this comment.
💡 Codex Review
After this commit moves the configuration article, this next-step link still targets /articles/Unlisted/Configure-Rillet, but that page no longer exists and docs/redirects.csv contains no redirect for it. Users following the link after connecting Rillet will receive a 404; update it to /articles/new-expensify/connections/rillet/Configure-Rillet or add the corresponding redirect.
ℹ️ 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".
HelpDot Documentation ReviewOverall AssessmentThis PR relocates two existing Rillet articles from docs/articles/Unlisted/ to a new public hub at docs/articles/new-expensify/connections/rillet/ and adds the hub landing page docs/new-expensify/hubs/connections/rillet.html. Both renames are 100% similarity moves, so no article body content changes in the diff. The scores below reflect the readiness of these articles to go live via this move, per the governance files. Scores Summary
Key Findings
Recommendations
Files Reviewed
Note: Detailed line-by-line feedback has been provided as inline comments. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Concierge reviewer checklist:
For more detailed instructions on completing this checklist, see How do I review a HelpDot PR as a Concierge Team member? cc @lschurr |
|
@mjasikowski @lschurr 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] NOTE: It looks like |
|
A preview of your ExpensifyHelp changes have been deployed to https://243f2165.helpdot.pages.dev ⚡️ Updated articles: |
|
@stephanieelliott - in the preview of both articles, there are images that don't load:
Should those load or is that normal for the preview? |
|
Oh @lschurr sorry I should've given you a heads up -- that's expected that the images don't load. Because they were also added in this PR, they won't render on the Dev build |
|
@MelvinBot the preview here shows the rillet hub and articles rendered correctly on helpdot.pages.dev. However, after merging this PR, the hub and articles are not showing on the live site help.expensify.com. Why is this happening, and how can I fix it? |
|
Good news: the Rillet hub and both articles are live and rendering correctly on help.expensify.com right now. Nothing is broken in the build — what you saw was a stale Cloudflare edge cache, and it has since cleared. I just verified on production:
Why it looked broken right after merge: HelpDot pages are served through Cloudflare with a 24-hour edge cache ( If you (or anyone) still see the old version: hard-refresh (Cmd/Ctrl+Shift+R) or wait for the edge cache to expire. No code change is needed for the hub or articles. One real follow-up to fix, though — there's a typo in one of the redirects this PR added. In I confirmed the old |
|
🚀 Deployed to staging by https://github.com/stephanieelliott in version: 9.4.61-0 🚀
|
|
🚀 Deployed to production by https://github.com/AndrewGable in version: 9.4.61-4 🚀
Bundle Size Analysis (Sentry): |


Explanation of Change
Unhide Rillet docs and create new hub folder in Connections section.
Fixed Issues
$
PROPOSAL:
Tests
Offline tests
N/A
QA Steps
Same as tests.
PR Author Checklist
### Fixed Issuessection aboveTestssectionOffline stepssectionQA stepssectiontoggleReportand 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