Add context that calls out disabling pay by invoice#24615
Add context that calls out disabling pay by invoice#24615akristen wants to merge 6 commits intodocker:mainfrom
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
/review |
![docker-agent[bot]](https://avatars.githubusercontent.com/in/2787757?s=60&v=4){kind=small}
There was a problem hiding this comment.
Assessment: 🟢 APPROVE
This PR appropriately expands the "Enable pay by invoice" section to cover both enabling and disabling the feature. The changes are clear, accurate, and follow Docker documentation standards.
Changes reviewed:
- Updated section heading to reflect both enable and disable functionality
- Added clarifying introductory sentence
- Expanded step 4 to provide specific instructions for both toggle states (Disabled → Enabled, Enabled → Disabled)
The documentation now provides complete guidance for users managing the pay by invoice feature in both directions.
No issues found.
…o add Link payment
|
/review |
…kdown goofiness, and my lazy writing
this drama.... |
|
/review |
craig-osterhout
left a comment
craig-osterhout
left a comment
There was a problem hiding this comment.
The agent got more nits than I would ever think of. LGTM
i know... i think she has it out for me sometimes... @craig-osterhout |
2 participants
Running this by support / billing before merging since i made an assumption about Link payment from my testing. still not clear what the exact prereq is to use this method....the doc says you have to have a card already filled out and that tracks with my testing, but i saw one instance where different UI copy appears in the payment portal...