feat: Add attendee custom form option #4480
Conversation
|
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/eventyay/open-event-frontend/300rd8hh1 |
Codecov Report
@@ Coverage Diff @@
## development #4480 +/- ##
===============================================
- Coverage 22.28% 22.26% -0.02%
===============================================
Files 469 470 +1
Lines 4950 4958 +8
Branches 6 6
===============================================
+ Hits 1103 1104 +1
- Misses 3846 3853 +7
Partials 1 1
Continue to review full report at Codecov.
|
|
It isn't even complete yet 🤷♂️ |
|
Means, your approach going in right direction.... |
|
There is no approach. I have literally just changed let to const |
iamareebjamal
force-pushed
the
attendee-custom-form
branch
from
13bb9dd to
885edd7
Compare
iamareebjamal
changed the title
|
Issues
======
- Added 4
Complexity increasing per file
==============================
- app/utils/text.ts 1
- app/components/forms/wizard/attendee-step.js 1
Complexity decreasing per file
==============================
+ app/models/custom-form.js -1
See the complete overview on Codacy |
| export function slugify(string: string, sep: string = '-'): string { | ||
| return string.toLowerCase() | ||
| .replace(/[^\w\s-]/g, '') // remove non-word [a-z0-9_], non-whitespace, non-hyphen characters | ||
| .replace(/[\s_-]+/g, sep) // swap any length of whitespace, underscore, hyphen characters with a single - |
There was a problem hiding this comment.
Codacy found an issue: Expected indentation of 4 spaces but found 8.
| @@ -0,0 +1,6 @@ | |||
| export function slugify(string: string, sep: string = '-'): string { | |||
| return string.toLowerCase() | |||
There was a problem hiding this comment.
Codacy found an issue: Expected indentation of 2 spaces but found 4.
| @@ -0,0 +1,6 @@ | |||
| export function slugify(string: string, sep: string = '-'): string { | |||
| return string.toLowerCase() | |||
| .replace(/[^\w\s-]/g, '') // remove non-word [a-z0-9_], non-whitespace, non-hyphen characters | |||
There was a problem hiding this comment.
Codacy found an issue: Expected indentation of 4 spaces but found 8.
| return string.toLowerCase() | ||
| .replace(/[^\w\s-]/g, '') // remove non-word [a-z0-9_], non-whitespace, non-hyphen characters | ||
| .replace(/[\s_-]+/g, sep) // swap any length of whitespace, underscore, hyphen characters with a single - | ||
| .replace(/^-+|-+$/g, ''); // remove leading, trailing - |
There was a problem hiding this comment.
Codacy found an issue: Expected indentation of 4 spaces but found 8.
Fixes #3199