Skip to content

Conversation

@vitaliiivanovspryker
Copy link
Collaborator

@vitaliiivanovspryker vitaliiivanovspryker commented Oct 31, 2025

…umentation

  • Updated dg_dev_sidebar.yml to include the "External HTTP requests" section in the performance guidelines.
  • Added a new markdown file with comprehensive guidelines for external HTTP requests, including caching, concurrent requests, and monitoring techniques.
  • Updated the performance guidelines overview to reference the new external HTTP requests section.

PR Description

Describe the context for your changes and the changes you've made.

Tickets

If changes are associated with a ticket, add a docs ticket here.

Checklist

  • I agree with the Code Contribution License Agreement in CONTRIBUTING.md
  • I validated and improved the content with Custom GPT
  • I checked my pages at the deployment preview website.
  • I invited a person to review and merge this PR.

@vitaliiivanovspryker vitaliiivanovspryker added the new PR contains a new document label Oct 31, 2025
@netlify
Copy link

netlify bot commented Oct 31, 2025

Deploy Preview for spryker-docs ready!

Name Link
🔨 Latest commit b3de844
🔍 Latest deploy log https://app.netlify.com/projects/spryker-docs/deploys/6904e014e69fb30008c9392e
😎 Deploy Preview https://deploy-preview-3331--spryker-docs.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 project configuration.

…umentation

- Updated `dg_dev_sidebar.yml` to include the "External HTTP requests" section in the performance guidelines.
- Added a new markdown file with comprehensive guidelines for external HTTP requests, including caching, concurrent requests, and monitoring techniques.
- Updated the performance guidelines overview to reference the new external HTTP requests section.
- Replaced outdated screenshot link with a hosted image URL.
- Removed the previously included local image file from the repository.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new PR contains a new document

Development

Successfully merging this pull request may close these issues.

2 participants