Skip to content

fix: correct domain for link column in org member table#17726

Merged
hbjORbj merged 1 commit intomainfrom
fix/org-member-link-safety
Nov 18, 2024
Merged

fix: correct domain for link column in org member table#17726
hbjORbj merged 1 commit intomainfrom
fix/org-member-link-safety

Conversation

@hbjORbj
Copy link
Copy Markdown
Contributor

@hbjORbj hbjORbj commented Nov 18, 2024

What does this PR do?

Screenshot 2024-11-18 at 1 16 45 PM

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • N/A - I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
  • I confirm automated tests are in place that prove my fix is effective or that my feature works.

@vercel
Copy link
Copy Markdown

vercel Bot commented Nov 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

2 Skipped Deployments
Name Status Preview Comments Updated (UTC)
cal ⬜️ Ignored (Inspect) Nov 18, 2024 6:15pm
calcom-web-canary ⬜️ Ignored (Inspect) Nov 18, 2024 6:15pm

@graphite-app graphite-app Bot requested a review from a team November 18, 2024 18:15
@dosubot dosubot Bot added organizations area: organizations, orgs 🐛 bug Something isn't working labels Nov 18, 2024
@keithwillcode keithwillcode added consumer core area: core, team members only labels Nov 18, 2024
@hbjORbj hbjORbj enabled auto-merge (squash) November 18, 2024 18:18
headers,
allMembers as UserTableUser[],
ATTRIBUTE_IDS,
domain
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is defined above:
const domain = orgBranding?.fullDomain ?? WEBAPP_URL;

@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Nov 18, 2024

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (11/18/24)

1 reviewer was added to this PR based on Keith Williams's automation.

@github-actions
Copy link
Copy Markdown
Contributor

E2E results are ready!

@hbjORbj hbjORbj merged commit 50df229 into main Nov 18, 2024
@hbjORbj hbjORbj deleted the fix/org-member-link-safety branch November 18, 2024 18:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐛 bug Something isn't working consumer core area: core, team members only organizations area: organizations, orgs ready-for-e2e

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants