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.

Sign up for GitHub

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

Jump to bottom

Polaris PSA certified connector #1004

Merged
merged 12 commits into from
Aug 16, 2021
Merged

Polaris PSA certified connector #1004

merged 12 commits into from
Aug 16, 2021

Conversation

repl-anand-vikram
Copy link
Contributor

@repl-anand-vikram repl-anand-vikram commented Aug 4, 2021 •
edited
Loading

When submitting a connector, please check the following conditions for your PR to ensure a smooth certification process.

  • 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 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.

@repl-anand-vikram repl-anand-vikram requested a review from a team as a code owner August 4, 2021 12:16
@ghost
Copy link

ghost commented Aug 4, 2021 •
edited by ghost
Loading

CLA assistant check
All CLA requirements met.

@sriyen-msft sriyen-msft added certified-connector needs:attention Needs attention from Microsoft or community labels Aug 6, 2021
sriyen-msft
sriyen-msft requested changes Aug 6, 2021
View reviewed changes
Copy link
Contributor

@sriyen-msft sriyen-msft left a comment

Choose a reason for hiding this comment

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

@repl-anand-vikram You need to sign CLA in the PR to get it approved and merged

sriyen-msft
sriyen-msft requested changes Aug 11, 2021
View reviewed changes
Copy link
Contributor

@sriyen-msft sriyen-msft left a comment

Choose a reason for hiding this comment

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

@repl-anand-vikram You need to sign CLA in the PR to get it approved and merged

certified-connectors/Polaris PSA/apiDefinition.swagger.json Outdated Show resolved Hide resolved
@sriyen-msft sriyen-msft added the Validation phase Connector is being validated by Microsoft. label Aug 11, 2021
@sriyen-msft sriyen-msft added needs:partner-attention and removed needs:attention Needs attention from Microsoft or community Validation phase Connector is being validated by Microsoft. labels Aug 12, 2021
@sriyen-msft
Copy link
Contributor

@repl-anand-vikram You need to sign CLA in the PR to get it approved and merged

sriyen-msft
sriyen-msft requested changes Aug 13, 2021
View reviewed changes
Copy link
Contributor

@sriyen-msft sriyen-msft left a comment

Choose a reason for hiding this comment

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

@repl-anand-vikram You need to sign CLA in the PR to get it approved and merged

@repl-anand-vikram
Copy link
Contributor Author

@repl-anand-vikram You need to sign CLA in the PR to get it approved and merged

@sriyen-msft we are reviewing the CLA with our legal team. while we do not see any concerns but we are waiting for formal approval. We will probably sign it on Monday. Thanks for following up

@repl-anand-vikram
Copy link
Contributor Author

@repl-anand-vikram You need to sign CLA in the PR to get it approved and merged

@sriyen-msft we are reviewing the CLA with our legal team. while we do not see any concerns but we are waiting for formal approval. We will probably sign it on Monday. Thanks for following up

@sriyen-msft I have signed the CLA. Please proceed with the review

@sriyen-msft sriyen-msft merged commit d52c830 into microsoft:dev Aug 16, 2021
joshuaapple pushed a commit to docjuris/PowerPlatformConnectors that referenced this pull request Oct 6, 2021
@repl-anand-vikram @joshuaapple
Polaris PSA certified connector (microsoft#1004)
a9108d4 
* Polaris PSA connector for Power Automate

* corrected base url description

* steps to obtain clientId client secret for oAuth flow

* correct formatting in readme.md

* corrected task api and security definition

* updated stack owner

* updated descriptions and added definition for date

* corrected description and added definition for Duration

* updated description

* updated icon brand color

* added getmytenantEndpoint details and changed definitions to set host from query

* corrected project details definition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sriyen-msft sriyen-msft sriyen-msft approved these changes

Assignees
No one assigned
Labels
certified-connector
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@repl-anand-vikram @sriyen-msft