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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃摗 Replace Provider Identifier with Integration Identifier #5061

Open
LetItRock opened this issue Jan 9, 2024 · 6 comments
Open

馃摗 Replace Provider Identifier with Integration Identifier #5061

LetItRock opened this issue Jan 9, 2024 · 6 comments
Assignees
Labels
good first issue Good for newcomers

Comments

@LetItRock
Copy link
Contributor

What?

On the integrations page, we show the provider identifier with each integration

image

image

image

Why? (Context)

It should be Integration Identifier because in code there is no any variable with name as providerIdentifier.
Instead we have integrationIdentifier variable at many places.
One such use case is to override email/sms provider. Here sendgrid-DqJizp2Zt is Integration Indentifier (or as per image Provider Identifier)

{
 "overrides": {
   "email": {
     "integrationIdentifier": "sendgrid-DqJizp2Zt"
   }
  }
}

Definition of Done

  • The Provider Identifier field name is replaced with Integration identifier field name in UI of an integration settings. See the mock Int. store
  • The Provider Identifier subtitle+value is replaced with Integration identifier value in the integrations list
    See the mock Int. store
  • The Provider Identifier hint is displayed on hover over Integration identifier subtitle.
  • The subtitle font is 12px size and 16px line height
  • The copy control is displayed on hover over Integration identifier subtitle
  • By click on identifier it is copied to the clipboard

See the mock below
Int. store

@LetItRock LetItRock added the good first issue Good for newcomers label Jan 9, 2024
Copy link

linear bot commented Jan 9, 2024

@AlexVCS
Copy link
Contributor

AlexVCS commented Jan 9, 2024

hi there, I'd like to work on this issue. can you assign it to me please?

@KaoutharAsma
Copy link

can I work on this ?

@animeshMondal-crypto
Copy link

It is not closed yet, so can i try out this issue??

@LetItRock
Copy link
Contributor Author

hey @AlexVCS! 馃憢 assigned this ticket to you ;)

@developerdhruv
Copy link

Want to work on it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants