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

Remove "/host/" from path #1314

Merged
merged 5 commits into from
Jan 4, 2022
Merged

Remove "/host/" from path #1314

merged 5 commits into from
Jan 4, 2022

Conversation

mingji1994
Copy link
Contributor

@mingji1994 mingji1994 commented Dec 8, 2021


When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)

  • I attest that the connector works and I verified by deploying and testing all the operations.
  • I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
  • I attest that I have added response schemas to my actions, unless the response schema is dynamic.
  • I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
  • If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

  • I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
  • Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
  • Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
  • If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

@mingji1994 mingji1994 requested a review from a team as a code owner December 8, 2021 02:31
@sriyen-msft sriyen-msft merged commit 55904cf into microsoft:dev Jan 4, 2022
robin-larsson pushed a commit to robin-larsson/PowerPlatformConnectors that referenced this pull request Jan 11, 2022
* Move hostUrl from connection parameter to operation parameter

* Move hostUrl from "path" to "header"

Move hostUrl from path to header, also change connectorParameters to headers.

* Remove "/host/" from path

* Update path
johannesh-recordedfuture pushed a commit to recordedfuture/PowerPlatformConnectors that referenced this pull request Mar 28, 2022
* Move hostUrl from connection parameter to operation parameter

* Move hostUrl from "path" to "header"

Move hostUrl from path to header, also change connectorParameters to headers.

* Remove "/host/" from path

* Update path
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.

3 participants