Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Dxp 1192 #390

Merged
merged 19 commits into from
Sep 12, 2022
Merged

Dxp 1192 #390

merged 19 commits into from
Sep 12, 2022

Conversation

BennyKitchell
Copy link
Contributor

@BennyKitchell BennyKitchell commented Sep 7, 2022 •
edited
Loading

Adds Dotnet code snippets to the docs

Checklist

  • Up to date with main
  • All the tests are passing
    • Delete all resources created in tests
  • Prettier
  • Spectral Lint
  • npm run bundle outputs nothing suspect
  • npm run postman outputs nothing suspect

Changes

Areas of Concern (optional)

jho44
jho44 suggested changes Sep 7, 2022
View reviewed changes
Copy link
Contributor

@jho44 jho44 left a comment

Choose a reason for hiding this comment

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

Missing Template Versions, Intl Autocomplete, Billing Groups, and Card Orders' snippets. Outside of that, LTGM. Thank you for taking this on.

@jho44 jho44 merged commit 3441819 into main Sep 12, 2022
@jho44 jho44 deleted the DXP-1192 branch September 12, 2022 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jho44 jho44 jho44 approved these changes

@sudoku-lord sudoku-lord Awaiting requested review from sudoku-lord

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@BennyKitchell @jho44 @sudoku-lord