Skip to content

fix: correct NetSuite capitalization and broken bold formatting in netsuite sync report#1670

Open
bhardwajdiv wants to merge 2 commits intohotwax:user-guides-pubfrom
bhardwajdiv:fix/netsuite-sync-report-typos
Open

fix: correct NetSuite capitalization and broken bold formatting in netsuite sync report#1670
bhardwajdiv wants to merge 2 commits intohotwax:user-guides-pubfrom
bhardwajdiv:fix/netsuite-sync-report-typos

Conversation

@bhardwajdiv
Copy link
Copy Markdown
Contributor

Summary
Fixes typos and formatting issues in the NetSuite sync reports page.

Changes

  • Fixed broken bold markdown: \*\*Filtering Return Transactions:\*8**Filtering Return Transactions:** in POS Returns vs Restock section
  • Fixed "Netsuite" → "NetSuite" in two section headings:
    • "Products without Netsuite Order ID Report"
    • "Customer Without Netsuite Id Report"

Page affected
https://docs.hotwax.co/documents/analytics/reports/netsuitesync

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the capitalization of "NetSuite" and corrects formatting in the NetSuite synchronization report documentation. The review identifies remaining instances of incorrect capitalization in subheadings at lines 63 and 162 and provides a suggestion to remove an extra space within a bolded label to ensure consistency with the style guide.

Comment thread documents/analytics/reports/netsuitesync.md
Comment thread documents/analytics/reports/netsuitesync.md Outdated
Copy link
Copy Markdown
Contributor

@dt2patel dt2patel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This still has unresolved review feedback on the formatting cleanup in the NetSuite sync report. Please address the remaining inline comment before merge.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@bhardwajdiv bhardwajdiv requested a review from dt2patel May 6, 2026 06:23
@bhardwajdiv
Copy link
Copy Markdown
Contributor Author

Thank you for your review. I have addressed the inline feedback by removing the extra space before the closing bold markers on line 430 in documents/analytics/reports/netsuitesync.md.

Kindly re-review it ,Sir.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants