Skip to content

feat(clerk-js): Display a callout when updating or removing a verified domain - #1641

Merged
panteliselef merged 5 commits into
mainfrom
ORG-176
Sep 1, 2023
Merged

feat(clerk-js): Display a callout when updating or removing a verified domain#1641
panteliselef merged 5 commits into
mainfrom
ORG-176

Conversation

@panteliselef

@panteliselef panteliselef commented Aug 28, 2023

Copy link
Copy Markdown
Contributor

Type of change

  • 🐛 Bug fix
  • 🌟 New feature
  • 🔨 Breaking change
  • 📖 Refactoring / dependency upgrade / documentation
  • other:

Packages affected

  • @clerk/clerk-js
  • @clerk/clerk-react
  • @clerk/nextjs
  • @clerk/remix
  • @clerk/types
  • @clerk/themes
  • @clerk/localizations
  • @clerk/clerk-expo
  • @clerk/backend
  • @clerk/clerk-sdk-node
  • @clerk/shared
  • @clerk/fastify
  • @clerk/chrome-extension
  • gatsby-plugin-clerk
  • build/tooling/chore

Description

  • npm test runs as expected.
  • npm run build runs as expected.

This PR

  • Introduces changes to CalloutWithAction (previously not used)
  • Display a callout in the enrolment and danger tabs of a verified domain
image image

@changeset-bot

changeset-bot Bot commented Aug 28, 2023

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 71fb33e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 0 packages

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@jit-ci jit-ci Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Great news! Jit hasn't found any security issues in your PR. Good Job! 🏆

@nikosdouvlis nikosdouvlis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Approved! I've left a few minor comments, good job @panteliselef :)

PS: There's a failing test blocking CICD

Comment thread packages/types/src/localization.ts Outdated
Comment thread packages/localizations/src/en-US.ts Outdated
Comment thread packages/clerk-js/src/ui/components/OrganizationProfile/VerifiedDomainPage.tsx Outdated
Comment thread packages/clerk-js/src/ui/components/OrganizationProfile/VerifiedDomainPage.tsx Outdated
Comment thread packages/clerk-js/src/ui/components/OrganizationProfile/VerifiedDomainPage.tsx Outdated
Comment thread packages/clerk-js/src/core/resources/OrganizationDomain.ts
@panteliselef panteliselef changed the title feat(clerk-js): Display a callout when updating or removing a verified domain [WIP] feat(clerk-js): Display a callout when updating or removing a verified domain Aug 29, 2023
@panteliselef panteliselef changed the title [WIP] feat(clerk-js): Display a callout when updating or removing a verified domain feat(clerk-js): Display a callout when updating or removing a verified domain Aug 30, 2023
Comment thread packages/clerk-js/src/ui/components/OrganizationProfile/VerifiedDomainPage.tsx Outdated
Comment thread packages/localizations/src/en-US.ts
@panteliselef
panteliselef force-pushed the ORG-176 branch 2 times, most recently from 21e0f1b to 5417c55 Compare August 31, 2023 12:39
@panteliselef

Copy link
Copy Markdown
Contributor Author

!snapshot

@clerk-cookie

Copy link
Copy Markdown
Collaborator

Hey @panteliselef - the snapshot version command generated the following package versions:

Package Version
@clerk/backend 0.28.0
@clerk/chrome-extension 0.3.30-snapshot.5417c55
@clerk/clerk-js 4.56.3-snapshot.5417c55
eslint-config-custom 0.3.0
@clerk/clerk-expo 0.18.21-snapshot.5417c55
@clerk/fastify 0.6.4
gatsby-plugin-clerk 4.4.5
@clerk/localizations 1.25.1-snapshot.5417c55
@clerk/nextjs 4.23.3
@clerk/clerk-react 4.24.0
@clerk/remix 2.10.0
@clerk/clerk-sdk-node 4.12.3
@clerk/shared 0.22.0
@clerk/themes 1.7.5
@clerk/types 3.50.0

Tip: use the snippet copy button below to quickly install the required packages.

# @clerk/backend
npm i @clerk/backend@0.28.0
# @clerk/chrome-extension
npm i @clerk/chrome-extension@0.3.30-snapshot.5417c55
# @clerk/clerk-js
npm i @clerk/clerk-js@4.56.3-snapshot.5417c55
# eslint-config-custom
npm i eslint-config-custom@0.3.0
# @clerk/clerk-expo
npm i @clerk/clerk-expo@0.18.21-snapshot.5417c55
# @clerk/fastify
npm i @clerk/fastify@0.6.4
# gatsby-plugin-clerk
npm i gatsby-plugin-clerk@4.4.5
# @clerk/localizations
npm i @clerk/localizations@1.25.1-snapshot.5417c55
# @clerk/nextjs
npm i @clerk/nextjs@4.23.3
# @clerk/clerk-react
npm i @clerk/clerk-react@4.24.0
# @clerk/remix
npm i @clerk/remix@2.10.0
# @clerk/clerk-sdk-node
npm i @clerk/clerk-sdk-node@4.12.3
# @clerk/shared
npm i @clerk/shared@0.22.0
# @clerk/themes
npm i @clerk/themes@1.7.5
# @clerk/types
npm i @clerk/types@3.50.0

@clerk-cookie

Copy link
Copy Markdown
Collaborator

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@clerk clerk locked as resolved and limited conversation to collaborators Sep 1, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants