Fix CA docs: ConsentPolicy, commandFor alignment, code formatting, style cleanup#4314
Fix CA docs: ConsentPolicy, commandFor alignment, code formatting, style cleanup#4314
Conversation
🚨🚨🚨 Docs migration in progress 🚨🚨🚨We are actively migrating UI extension reference docs to MDX in the
During this migration, please be aware of the following:
Doc comments in Examples that previously lived in this repo are being moved to the What should I do?
Thanks for your patience while we complete the migration! 🙏 |
|
We detected some changes in |
mcvinci
left a comment
There was a problem hiding this comment.
Thanks, @sordaz00! Same suggestions as here: #4317 (review)
…yle cleanup - Add description + @publicdocs to ConsentPolicy type - Align commandFor/command descriptions across docs.ts, MapMarker.d.ts, components.d.ts - Fix passive voice on DatePicker allow/allowedDays props - Add backticks on numeric values in Progress value/max descriptions - Fix ChoiceList values prop wonky backtick pluralization - Convert remaining @see tags to inline links in CA docs.ts, shared.ts, order-status.ts - Remove "Note that" filler from PhoneField, DatePicker, Sheet descriptions - Fix "buyer" -> "user" in DatePicker onInvalid - Fix onClick missing backtick in components.d.ts Made-with: Cursor
3a98996 to
b5fe0bc
Compare
Summary
@publicDocstoConsentPolicytypecommandFor/commanddescriptions acrossdocs.ts,MapMarker.d.ts,components.d.tsallow/allowedDayspropsvalue/maxdescriptionsvaluesprop wonky backtick pluralization@seetags to inline links in CAdocs.ts,shared.ts,order-status.tsonInvalidonClickmissing backtick incomponents.d.tsCloses: https://github.com/shop/issues-learn/issues/1691
Closes: https://github.com/shop/issues-learn/issues/1693
Closes: https://github.com/shop/issues-learn/issues/1697
Closes: https://github.com/shop/issues-learn/issues/1701
Closes: https://github.com/shop/issues-learn/issues/1702
Closes: https://github.com/shop/issues-learn/issues/1708
Closes: https://github.com/shop/issues-learn/issues/1712
Closes: https://github.com/shop/issues-learn/issues/1715
Test plan
yarn buildpassesMade with Cursor