Skip to content

Commit

Permalink
fix: remove subdenom display check (#221)
Browse files Browse the repository at this point in the history
  • Loading branch information
fmorency authored Jan 31, 2025
1 parent d041b41 commit 75fc3c5
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 14 deletions.
8 changes: 1 addition & 7 deletions components/factory/forms/TokenDetailsForm.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,13 +18,7 @@ export default function TokenDetails({
address: string;
}>) {
const TokenDetailsSchema = Yup.object().shape({
display: Yup.string()
.required('Display is required')
.noProfanity()
.test('display-contains-subdenom', 'Display must contain subdenom', function (value) {
const subdenom = this.parent.subdenom;
return !subdenom || value.toLowerCase().includes(subdenom.slice(1).toLowerCase());
}),
display: Yup.string().required('Display is required').noProfanity(),
description: Yup.string()
.required('Description is required')
.min(10, 'Description must be at least 10 characters long')
Expand Down
8 changes: 1 addition & 7 deletions components/factory/modals/updateDenomMetadata.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -15,13 +15,7 @@ import { useProposalsByPolicyAccount } from '@/hooks/useQueries';

const TokenDetailsSchema = (context: { subdenom: string }) =>
Yup.object().shape({
display: Yup.string()
.required('Display is required')
.noProfanity()
.test('display-contains-subdenom', 'Display must contain subdenom', function (value) {
const subdenom = context.subdenom;
return !subdenom || value.toLowerCase().includes(subdenom.slice(1).toLowerCase());
}),
display: Yup.string().required('Display is required').noProfanity(),
name: Yup.string().required('Name is required').noProfanity(),
description: Yup.string()
.required('Description is required')
Expand Down

0 comments on commit 75fc3c5

Please sign in to comment.