Skip to content

EDW correcting endpoint in documentation#66

Merged
TomerPacific merged 1 commit intomasterfrom
feature/fix-endpoint-urls-in-documentation
Nov 8, 2025
Merged

EDW correcting endpoint in documentation#66
TomerPacific merged 1 commit intomasterfrom
feature/fix-endpoint-urls-in-documentation

Conversation

@TomerPacific
Copy link
Copy Markdown
Owner

Resolves #64

This pull request updates the data source used in the example for EnhancedDropDown.withEndpoint in the README.md file. The URL for fetching dropdown data now points to a JSON file hosted on GitHub instead of the previous mock API.

  • Documentation update:
    • Changed the urlToFetchData example in the README.md to use a GitHub-hosted JSON file instead of a Mocky API endpoint.

@TomerPacific TomerPacific requested a review from Copilot November 8, 2025 06:45
@TomerPacific TomerPacific self-assigned this Nov 8, 2025
@TomerPacific TomerPacific added invalid This doesn't seem right documentation Knowledge is power labels Nov 8, 2025
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the README example code to use a real GitHub raw content endpoint instead of a mock API service for fetching dropdown data.

  • Replaces the mocky.io mock endpoint with a GitHub raw content URL pointing to the actual example file in the repository
  • Improves code formatting by splitting the URI constructor across multiple lines for better readability

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@TomerPacific TomerPacific merged commit 246664a into master Nov 8, 2025
7 checks passed
@TomerPacific TomerPacific deleted the feature/fix-endpoint-urls-in-documentation branch November 8, 2025 06:48
@TomerPacific TomerPacific mentioned this pull request Nov 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Knowledge is power invalid This doesn't seem right

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Correct example in documentation

2 participants