-
Notifications
You must be signed in to change notification settings - Fork 3
Fix/youl bug fix #150
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.
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
Fix/youl bug fix #150
Changes from 14 commits
6b7ea7c
3257b25
536f3b2
17d4514
59ac630
e0e83ee
0878802
ff82386
f8f4f95
52eb474
013fde3
8ecaabf
fef8165
fea5c28
de86036
4dc9e50
3ce5331
cbb4d4e
2656c9c
43a4179
43fd284
9260be9
96f1e22
0a5ab41
5e014cf
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -88,9 +88,7 @@ export default function CreateGatheringForm({ | |
| /> | ||
| </div> | ||
| <div className="flex flex-col gap-3"> | ||
| <label htmlFor="gathering-image" className="text-base font-semibold text-gray-800"> | ||
| μ΄λ―Έμ§ μ ν/μ²¨λΆ | ||
| </label> | ||
| <span className="text-base font-semibold text-gray-800">μ΄λ―Έμ§ μ ν/첨λΆ</span> | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. μ κ·Όμ± κ°μ μ μν΄ μλ―Έλ‘ μ νκ·Έ μ¬μ© νμ μ΄λ―Έμ§ μ ν λ μ΄λΈμ΄ λ€μκ³Ό κ°μ΄ μμ νλ κ²μ μ μν©λλ€: -<span className="text-base font-semibold text-gray-800">μ΄λ―Έμ§ μ ν/첨λΆ</span>
+<label htmlFor="gathering-image" className="text-base font-semibold text-gray-800">μ΄λ―Έμ§ μ ν/첨λΆ</label>
|
||
| <div className="flex"> | ||
| <Controller | ||
| name="imageUrl" | ||
|
|
@@ -161,9 +159,7 @@ export default function CreateGatheringForm({ | |
| </div> | ||
| <div className="flex flex-col gap-3"> | ||
| <div className="flex justify-between"> | ||
| <label htmlFor="gathering-dateTime" className="text-base font-semibold text-gray-800"> | ||
| λ μ§ | ||
| </label> | ||
| <span className="text-base font-semibold text-gray-800">λ μ§</span> | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. λ μ§ μ ν λ μ΄λΈμ μλ―Έλ‘ μ λ§ν¬μ κ°μ νμ λ μ§ μ ν λ μ΄λΈλ λ€μκ³Ό κ°μ΄ μμ νλ κ²μ μ μν©λλ€: -<span className="text-base font-semibold text-gray-800">λ μ§</span>
+<label htmlFor="gathering-datetime" className="text-base font-semibold text-gray-800">λ μ§</label>
|
||
| </div> | ||
| <Controller | ||
| name="dateTime" | ||
|
|
@@ -223,6 +219,7 @@ export default function CreateGatheringForm({ | |
| render={({ field }) => ( | ||
| <Textarea | ||
| {...field} | ||
| id="gathering-introduce" | ||
| placeholder="λͺ¨μ§ μ€λͺ /곡μ§λ₯Ό 100μ μ΄λ΄λ‘ μ λ ₯ν΄μ£ΌμΈμ." | ||
| maxLength={100} | ||
| inputClassNames="h-40 py-2.5 px-4 bg-gray-100 placeholder:text-gray-400 font-pretendard text-base font-medium rounded-xl" | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,13 +6,14 @@ import { useGetHostedGatheringListQuery } from '@/src/_queries/my-gathering/host | |
| import { formatDateToRequest } from '@/src/utils/format-date'; | ||
| import GatheringListWithDate from '@/src/app/(crew)/my-gathering/_component/gathering-list-with-date'; | ||
| import PopOverCalendar from '@/src/components/common/input/pop-over-calendar'; | ||
| import MyGatheringSkeletonList from '@/src/components/common/skeleton/my-gathering-skeleton-list'; | ||
| import { GatheringCardProps } from '@/src/types/gathering-data'; | ||
|
|
||
| export default function MyGatheringHostedPage() { | ||
| const [selectedDate, setSelectedDate] = useState(new Date()); | ||
| const [hostedGatheringList, setHostedGatheringList] = useState<GatheringCardProps[]>(); | ||
|
|
||
| const { data, refetch } = useQuery( | ||
| const { data, isLoading, refetch } = useQuery( | ||
| useGetHostedGatheringListQuery(formatDateToRequest(selectedDate)), | ||
| ); | ||
|
|
||
|
|
@@ -29,6 +30,7 @@ export default function MyGatheringHostedPage() { | |
| <div className="py-4 md:py-6"> | ||
| <PopOverCalendar value={selectedDate} onChange={(d) => setSelectedDate(d)} /> | ||
| </div> | ||
| {isLoading && <MyGatheringSkeletonList num={6} />} | ||
| {hostedGatheringList && <GatheringListWithDate gatheringList={hostedGatheringList} />} | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. π οΈ Refactor suggestion λ‘λ© μν UIκ° μ μ ν ꡬνλμμ΅λλ€. μ€μΌλ ν€ λ‘λ© μνμ λ°μ΄ν° νμκ° μ‘°κ±΄λΆ λ λλ§μΌλ‘ μ ꡬνλμμ΅λλ€. νμ§λ§ μλ¬ μν μ²λ¦¬κ° λλ½λμ΄ μμ΅λλ€. λ€μκ³Ό κ°μ΄ μλ¬ μν μ²λ¦¬λ₯Ό μΆκ°νλ κ²μ μΆμ²λ립λλ€: {isLoading && <MyGatheringSkeletonList num={6} />}
+ {error && <div className="text-red-500">λ°μ΄ν°λ₯Ό λΆλ¬μ€λ μ€ μ€λ₯κ° λ°μνμ΅λλ€.</div>}
{hostedGatheringList && <GatheringListWithDate gatheringList={hostedGatheringList} />}
|
||
| </div> | ||
| ); | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
π οΈ Refactor suggestion
μ΄λ―Έμ§ μ ν UIμ μλ§¨ν± λ§ν¬μ κ°μ μ΄ νμν©λλ€.
labelμ리먼νΈλ₯ΌspanμΌλ‘ λ³κ²½νλ©΄ μ κ·Όμ±μ΄ μ νλ μ μμ΅λλ€.λ€μκ³Ό κ°μ΄ μμ νλ κ²μ μ μλ립λλ€: