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

feat: Address issues with packages #47

Merged
merged 3 commits into from
Sep 2, 2024
Merged

feat: Address issues with packages #47

merged 3 commits into from
Sep 2, 2024

Conversation

ductaily
Copy link
Contributor

@ductaily ductaily commented Aug 28, 2024

  • → the namespace was wrong and the package name shall be package.name.replace(/@/,'').replace(/\//g,'.')
  • "partOfProducts": [ "customer:product:capire.bookstore:” ] → has to be the same as the product name in the products section
  • "vendor": "customer:vendor:customer:” → has to be the same as the vendor in the products section
  • "title": "capire bookshop” → take what is there, don’t add extra text like “sample title for"
  • “shortDescription" → take what is there, don’t add extra text like “Here’s the short description for ..."
  • “description" → take what is there, don’t add extra text like “Here’s the short description for ..."

In #38

@ductaily
Copy link
Contributor Author

If I understood correctly the first bullet point was addressed in the PR for the Products
→ the namespace was wrong and the package name shall be package.name.replace(/@/,'').replace(/\//g,'.')

Is my understanding correct? @RoshniNaveenaS

CC: @aramovic79 @Fannon

Co-authored-by: Simon Heimler <[email protected]>
@RoshniNaveenaS
Copy link
Contributor

d in the PR for the Prod

Yes we had the fix in this PR

@ductaily
Copy link
Contributor Author

d in the PR for the Prod

Yes we had the fix in this PR

Thank you for the confirmation.

@ductaily ductaily marked this pull request as ready for review August 30, 2024 11:20
@ductaily ductaily requested a review from Fannon August 30, 2024 11:20
Copy link
Contributor

@RoshniNaveenaS RoshniNaveenaS left a comment

Choose a reason for hiding this comment

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

Looks good to me

@ductaily ductaily merged commit 33fb693 into main Sep 2, 2024
2 checks passed
@ductaily ductaily deleted the feat/address-packages branch September 2, 2024 07:03
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.

3 participants