Skip to content

feat: add email domain to the segment identify call #1680

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
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

feat: add email domain to the segment identify call #1680

wants to merge 1 commit into from

Conversation

@mfrancisc
Copy link
Contributor

  • get the user email from the signup response
  • extract email domain from the user email
  • send only email domain with the segment identify call

wrt: https://issues.redhat.com/browse/SANDBOX-1439

depends on codeready-toolchain/registration-service#555

@rhdh-gh-app
Copy link

rhdh-gh-app bot commented Oct 20, 2025

Changed Packages

Package Name Package Path Changeset Bump Current Version
@red-hat-developer-hub/backstage-plugin-sandbox workspaces/sandbox/plugins/sandbox none v0.1.0

@sonarqubecloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Lucifergene Lucifergene Awaiting requested review from Lucifergene Lucifergene will be requested when the pull request is marked ready for review Lucifergene is a code owner

@rohitkrai03 rohitkrai03 Awaiting requested review from rohitkrai03 rohitkrai03 will be requested when the pull request is marked ready for review rohitkrai03 is a code owner

@karthikjeeyar karthikjeeyar Awaiting requested review from karthikjeeyar karthikjeeyar will be requested when the pull request is marked ready for review karthikjeeyar is a code owner

@xcoulon xcoulon Awaiting requested review from xcoulon xcoulon will be requested when the pull request is marked ready for review xcoulon is a code owner

@alexeykazakov alexeykazakov Awaiting requested review from alexeykazakov alexeykazakov will be requested when the pull request is marked ready for review alexeykazakov is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

do-not-merge/work-in-progress

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mfrancisc