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

1Password codeless connector #11131

Merged
merged 6 commits into from
Oct 22, 2024
Merged

Conversation

azurekid
Copy link
Contributor

Required items, please complete

Change(s):

  • Added Codeless connector

Reason for Change(s):

  • Request for connector without PaaS components

Version Updated:

  • N/A
    Testing Completed:
  • See guidance below

@azurekid azurekid requested review from a team as code owners September 17, 2024 11:21
@azurekid
Copy link
Contributor Author

Can you please help to get the new connector to be recognized by the Create-Solution script as it is now not added to the existing content package.

@scottisloud FYI, I have created a codeless connector for 1Password

@v-prasadboke v-prasadboke self-assigned this Sep 18, 2024
@v-prasadboke v-prasadboke added Connector Connector specialty review needed Solution Solution specialty review needed Codeless Connector Platform (CCP) Connector labels Sep 18, 2024
@azurekid azurekid changed the title added codeless connector 1Password codeless connector Sep 23, 2024
Copy link
Contributor

Choose a reason for hiding this comment

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

Hello @azurekid, DCR is missing with columns. Is it ok ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

hi @v-prasadboke,
It is correct that the columns are not defined in the DCR.
The way the DCR is now configured, all data will be sent through to the LA workspace.
This is also how the DCR is configured in the previous 1Password solution.

adding @scottisloud for visibility at 1Password

@v-prasadboke v-prasadboke merged commit b625b86 into Azure:master Oct 22, 2024
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Codeless Connector Platform (CCP) Connector Connector Connector specialty review needed Solution Solution specialty review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants