Skip to content

refactor: enhance Dataverse hostname validation and clean up connecti…#573

Open
cyco77 wants to merge 2 commits into
PowerPlatformToolBox:devfrom
cyco77:feature/api-crm
Open

refactor: enhance Dataverse hostname validation and clean up connecti…#573
cyco77 wants to merge 2 commits into
PowerPlatformToolBox:devfrom
cyco77:feature/api-crm

Conversation

@cyco77

@cyco77 cyco77 commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Please fill in this template.

  • Use a meaningful title for the pull request.
  • Follow the guidelines from the CONTRIBUTING.md.
  • Mention the bug or the feature number the PR will be targeting.
  • Test the change in your own code. (Compile and run)
  • Resolve all GH Copilot comments.

@Power-Maverick

Copy link
Copy Markdown
Contributor

@cyco77 is this resolving #561 ? Does auth works on api.crm and all the dataverse methods might not work either. This needs to be looked at in-depth unless you have done your due diligence.

@cyco77

cyco77 commented Jun 23, 2026

Copy link
Copy Markdown
Contributor Author

I already checked some of the features, but will have an additional look into it...

@cyco77

cyco77 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

I double checked several tools and all works fine. Double check documentation as well and it's recommended to use this endpoint for api requests: https://learn.microsoft.com/en-us/power-apps/developer/data-platform/webapi/web-api-service-documents?tabs=insomnia

@Power-Maverick

Copy link
Copy Markdown
Contributor

@cyco77 when I use interactive mode I get error (see below)

image

@Power-Maverick Power-Maverick added discussion needed Link a discussion on this topic pr-in-review regression-test This item will need thorough regression test labels Jun 24, 2026
@cyco77

cyco77 commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

will have a look on it

@MattBerg11

Copy link
Copy Markdown
Collaborator

@cyco77 could we wrap this this issue in with this as well?

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

Labels

discussion needed Link a discussion on this topic regression-test This item will need thorough regression test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants