Skip to content

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

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

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?

@cyco77 cyco77 closed this Jul 21, 2026
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