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.

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

Add stock_transfers type in ResourceListItem component #423

Merged
merged 8 commits into from
Nov 9, 2023

Conversation

pfferrari
Copy link
Contributor

@pfferrari pfferrari commented Nov 9, 2023

What I did

I added stock_transfers related dictionary file and enabled ResourceListItem to render resources of kind stock_transfers.

Screenshot 2023-11-09 alle 15 55 03

I also updated the rendering of resources of kind returns to match updated design.

Screenshot 2023-11-09 alle 15 56 25

Checklist

  • Make sure your changes are tested (stories and/or unit, integration, or end-to-end tests).
  • Make sure to add/update documentation regarding your changes.
  • You are NOT deprecating/removing a feature.

Copy link

netlify bot commented Nov 9, 2023

Deploy Preview for commercelayer-app-elements ready!

Name Link
🔨 Latest commit 9903fa2
🔍 Latest deploy log https://app.netlify.com/sites/commercelayer-app-elements/deploys/654d14aed707700008610562
😎 Deploy Preview https://deploy-preview-423--commercelayer-app-elements.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@pfferrari pfferrari self-assigned this Nov 9, 2023
@pfferrari pfferrari added the enhancement New feature or request label Nov 9, 2023
@pfferrari pfferrari changed the title Add stock_transfers type in ResourceListItems component Add stock_transfers type in ResourceListItem component Nov 9, 2023
Copy link
Contributor

Choose a reason for hiding this comment

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

shall we keep those file names snakeCase or 1:1 as the resource name?

Copy link
Contributor

Choose a reason for hiding this comment

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

I would use the camelCase notation: stockTransfers.tsx

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure! I just renamed both the dictionary and transformer file! :)

@pfferrari pfferrari requested a review from gciotola November 9, 2023 17:20
@pfferrari pfferrari merged commit 67b6a94 into main Nov 9, 2023
4 checks passed
@pfferrari pfferrari deleted the add-stock-transfers-list-item branch November 9, 2023 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants