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

DDSaaS: Shopify: Crawler Integration v1.0.0 #18692

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

akaila-crest
Copy link
Contributor

What does this PR do?

This is an initial release PR of Shopify integration including all the required assets.

Additional Notes

  • Crawler code for this integration has been committed in its respective repo
  • Pipeline created for this integration are available in our sandbox and would be shared separately with the required teams.
  • Samples for the pipeline review would also be shared separately with the required teams.
  • Since during the standard attribute remapping we are not preserving the source attributes as per suggested best practices, it would result in filters using these standard attributes populating the values of other integrations as well as per current Datadog behavior.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Changelog entries must be created for modifications to shipped code
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

cmlaverdiere
cmlaverdiere previously approved these changes Nov 18, 2024
@cmlaverdiere
Copy link
Contributor

cmlaverdiere commented Nov 18, 2024

@akaila-crest Is this ready for review? If so please remove the PR from draft status. Also please resolve codeowners conflict.

@akaila-crest akaila-crest marked this pull request as ready for review November 18, 2024 10:54
@akaila-crest akaila-crest requested review from a team as code owners November 18, 2024 10:54
@akaila-crest
Copy link
Contributor Author

@cmlaverdiere, I have resolved the conflict, and yes PR is ready for the review.

@hestonhoffman hestonhoffman added the editorial review Waiting on a more in-depth review from a docs team editor label Nov 18, 2024
Copy link
Contributor

@estherk15 estherk15 left a comment

Choose a reason for hiding this comment

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

Left an initial review for documentation, let me know if you have any questions!


[Shopify][1] is a comprehensive commerce platform designed to help individuals start, manage, and grow their businesses. It provides tools to build an online store, manage sales, market to customers, and accept payments in both digital and physical locations.

The Shopify Integration collects Event, Product, Customer and Order logs, sending them to Datadog for detailed analysis.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is "Customer and Order" one type of log or two separate types?

Suggested change
The Shopify Integration collects Event, Product, Customer and Order logs, sending them to Datadog for detailed analysis.
The Shopify Integration collects Event, Product, Customer, and Order logs, sending them to Datadog for detailed analysis.

Choose a reason for hiding this comment

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

Hi @estherk15,
Customer and Order are 2 separate log types.

Comment on lines +16 to +18
1. Log in to [Shopify][2] admin account
2. Shopify Store name is the `xxxx` part of the Store URL (`https://admin.shopify.com/store/xxxx`)
3. Navigate to **Settings > Apps and sales channels**.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
1. Log in to [Shopify][2] admin account
2. Shopify Store name is the `xxxx` part of the Store URL (`https://admin.shopify.com/store/xxxx`)
3. Navigate to **Settings > Apps and sales channels**.
1. Log in to [Shopify][2] admin account.
2. The Shopify Store name is the `xxxx` part of the Store URL (`https://admin.shopify.com/store/xxxx`).
3. Navigate to **Settings > Apps and sales channels**.

1. Log in to [Shopify][2] admin account
2. Shopify Store name is the `xxxx` part of the Store URL (`https://admin.shopify.com/store/xxxx`)
3. Navigate to **Settings > Apps and sales channels**.
4. Select **Develop apps** and Click **Allow custom app development**.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
4. Select **Develop apps** and Click **Allow custom app development**.
4. Select **Develop apps** and click **Allow custom app development**.

2. Shopify Store name is the `xxxx` part of the Store URL (`https://admin.shopify.com/store/xxxx`)
3. Navigate to **Settings > Apps and sales channels**.
4. Select **Develop apps** and Click **Allow custom app development**.
5. Click **Create a custom app**, provide the necessary details and Click on **Create app**.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
5. Click **Create a custom app**, provide the necessary details and Click on **Create app**.
5. Click **Create a custom app**, provide the necessary details and click **Create app**.

- **read_customers**
- **read_content**
- **read_price_rules**
8. Click on **Save** to apply the changes.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
8. Click on **Save** to apply the changes.
8. Click **Save** to apply the changes.

"id": 8602455036825926,
"definition": {
"type": "note",
"content": "\n**[Shopify](https://www.shopify.com/)** is a comprehensive commerce platform that help individuals to start, manage, and grow a business. It offers a set of tools to build an online store, manage sales, market to customers, and accept payments in digital and physical locations.\n\nThe **Shopify Orders Overview** Dashboard provides an overview of orders within your store. It provides insights into order tracking, fulfillments, customers and product details.\n\nFor more information, see the [Shopify Integration Documentation](https://docs.datadoghq.com/integrations/shopify/).\n\n### Tip:-\n- Clone this dashboard to rearrange, modify and add widgets and visualizations.\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"content": "\n**[Shopify](https://www.shopify.com/)** is a comprehensive commerce platform that help individuals to start, manage, and grow a business. It offers a set of tools to build an online store, manage sales, market to customers, and accept payments in digital and physical locations.\n\nThe **Shopify Orders Overview** Dashboard provides an overview of orders within your store. It provides insights into order tracking, fulfillments, customers and product details.\n\nFor more information, see the [Shopify Integration Documentation](https://docs.datadoghq.com/integrations/shopify/).\n\n### Tip:-\n- Clone this dashboard to rearrange, modify and add widgets and visualizations.\n",
"content": "\n**[Shopify](https://www.shopify.com/)** is a comprehensive commerce platform that helps individuals to start, manage, and grow a business. It offers a set of tools to build an online store, manage sales, market to customers, and accept payments in digital and physical locations.\n\nThe **Shopify Orders Overview** Dashboard provides an overview of orders within your store. It provides insights into order tracking, fulfillments, customers and product details.\n\nFor more information, see the [Shopify Integration Documentation](https://docs.datadoghq.com/integrations/shopify/).\n\n### Tip:-\n- Clone this dashboard to rearrange, modify, and add widgets and visualizations.\n",

"id": 1767411351951850,
"definition": {
"type": "note",
"content": "**[Shopify](https://www.shopify.com/)** is a comprehensive commerce platform that help individuals to start, manage, and grow a business. It offers a set of tools to build an online store, manage sales, market to customers, and accept payments in digital and physical locations.\n\nThe **Shopify Product Overview** Dashboard provides an overview of products within your store. It provides insights into product types, status distribution, inventory and variant details.\n\nFor more information, see the [Shopify Integration Documentation](https://docs.datadoghq.com/integrations/shopify/).\n\n### Tip:-\n- Clone this dashboard to rearrange, modify and add widgets and visualizations.\n",
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"content": "**[Shopify](https://www.shopify.com/)** is a comprehensive commerce platform that help individuals to start, manage, and grow a business. It offers a set of tools to build an online store, manage sales, market to customers, and accept payments in digital and physical locations.\n\nThe **Shopify Product Overview** Dashboard provides an overview of products within your store. It provides insights into product types, status distribution, inventory and variant details.\n\nFor more information, see the [Shopify Integration Documentation](https://docs.datadoghq.com/integrations/shopify/).\n\n### Tip:-\n- Clone this dashboard to rearrange, modify and add widgets and visualizations.\n",
"content": "**[Shopify](https://www.shopify.com/)** is a comprehensive commerce platform that helps individuals to start, manage, and grow a business. It offers a set of tools to build an online store, manage sales, market to customers, and accept payments in digital and physical locations.\n\nThe **Shopify Product Overview** Dashboard provides an overview of products within your store. It provides insights into product types, status distribution, inventory and variant details.\n\nFor more information, see the [Shopify Integration Documentation](https://docs.datadoghq.com/integrations/shopify/).\n\n### Tip:-\n- Clone this dashboard to rearrange, modify, and add widgets and visualizations.\n",

"created_at": "2024-10-01",
"last_updated_at": "2024-10-01",
"title": "Order Cancellation Rate is High",
"description": "High order cancellation rate is monitored.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add more context, following the best practices guide:

  • This is not a copy of the title.
  • Define the problem stated by the title.
  • Answer why this is an issue worth alerting on.
  • Describe the impact of the problem.

"created_at": "2024-10-01",
"last_updated_at": "2024-10-01",
"title": "Product Inventory is Out of Stock",
"description": "Out of stock product inventory is monitored.",
Copy link
Contributor

Choose a reason for hiding this comment

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

Can you add more context, following the best practices guide:

  • This is not a copy of the title.
  • Define the problem stated by the title.
  • Answer why this is an issue worth alerting on.
  • Describe the impact of the problem.

"manifest_version": "2.0.0",
"app_uuid": "81c0f478-e722-454a-83d3-5e3f45e11ca8",
"app_id": "shopify",
"display_on_public_website": false,
Copy link
Contributor

Choose a reason for hiding this comment

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

To confirm, the docs are only available in the integration tile and not on the public documentation site.

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

Successfully merging this pull request may close these issues.

6 participants