[Admin] Component: Create event details - UI component #214 #293

KYJKY · 2024-09-01T11:28:05Z

Task

#214

작업사항

NewEventDetailsComponent.razor 생성: 어드민 페이지에서 이벤트를 생성하는 컴포넌트를 만들었습니다.
Model 클래스 폴더 생성: 이벤트 생성 데이터를 담기 위해 기존 API 모델 클래스 복사

궁금한/잘 모르겠는 부분

컴포넌트 단위에 대한 고민: 컴포넌트 계층구조에 대한 고민이 있습니다. 특히 제가 작업한 내용이 어떤 외부 컴포넌트에서 참조하는지에 대해 불분명?한 상황입니다. /Components/Pages/AdminCreateEvent.razor와 같은 외부 컴포넌트 내에 제가 작업한 NewEventDetailsComponent.razor를 포함시키는 방식으로 만들어야 하는지, 아니면 현재 만들어져 있는 /Components/Pages/Admin.razor에서 추후 참조하게 만들게 될지에 대해 잘 모르겠습니다.
Task 단위에 대한 고민: 해당 Task와 [Admin] Component: Create event details - UI logic #215 Task와의 경계가 궁금합니다. 아래 작업들이 현재 Task에서 진행되어야 할지, 215번에서 진행되어야 할지 궁금합니다.
- Front App에서 유효성검사
- 입력값을 Model Class에 맵핑
CSS 관리에 대해 내부에 <Style>로 작성해도 상관 없을지 궁금합니다.
단위 테스트 코드에 대하여 각각 모든 input에 대해 존재 여부를 메서드로 추가하면 충분한 테스트인지 궁금합니다. 만약 유효성 검사 코드를 추가하게 된다면 input 단위로 유효성 검사 코드를 테스트 하는 메서드를 각각 추가하면 될까요?

추후 작업 예정

단위 테스트 코드 작성
Task 단위에 따라 유효성 검사 및 Model Class 맵핑 작업 추가

justinyoo · 2024-09-01T14:05:54Z

@KYJKY 아무래도 새 페이지 /admin/events/new를 만들어야 하는 시점이 됐네요. 이 페이지 먼저 만들어 볼까요? 새 태스크 열어두겠습니다. 그것 먼저 작업하시지요. 파일 이름은 Components/Pages/AdminNewEvent.razor 쯤으로 하겠습니다.

justinyoo · 2024-09-01T14:08:07Z

#295

justinyoo

컴포넌트 단위에 대한 고민: 컴포넌트 계층구조에 대한 고민이 있습니다. 특히 제가 작업한 내용이 어떤 외부 컴포넌트에서 참조하는지에 대해 불분명?한 상황입니다. /Components/Pages/AdminCreateEvent.razor와 같은 외부 컴포넌트 내에 제가 작업한 NewEventDetailsComponent.razor를 포함시키는 방식으로 만들어야 하는지, 아니면 현재 만들어져 있는 /Components/Pages/Admin.razor에서 추후 참조하게 만들게 될지에 대해 잘 모르겠습니다.

AdminNewEvent.razor 페이지를 만들었습니다. 그 안에 넣어두세요.

Task 단위에 대한 고민: 해당 Task와 #215 Task와의 경계가 궁금합니다. 아래 작업들이 현재 Task에서 진행되어야 할지, 215번에서 진행되어야 할지 궁금합니다.

Front App에서 유효성검사
-입력값을 Model Class에 맵핑

이 PR에서는 다루지 않고 #215 에서 진행합니다.

CSS 관리에 대해 내부에 <Style>로 작성해도 상관 없을지 궁금합니다.

Scoped CSS로 작성하세요. 하지만, 그 이전에 기본 Fluent UI 제공 CSS로 충분한지 먼저 검토해 보세요.

단위 테스트 코드에 대하여 각각 모든 input에 대해 존재 여부를 메서드로 추가하면 충분한 테스트인지 궁금합니다.

맞습니다.

만약 유효성 검사 코드를 추가하게 된다면 input 단위로 유효성 검사 코드를 테스트 하는 메서드를 각각 추가하면 될까요?

#215 에서 진행합니다.

AzureOpenAIProxy.sln

src/AzureOpenAIProxy.PlaygroundApp/Components/UI/Admin/NewEventDetailsComponent.razor

src/AzureOpenAIProxy.PlaygroundApp/Models/AdminEventDetails.cs

src/AzureOpenAIProxy.PlaygroundApp/Models/EventDetails.cs

test/AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsTests.cs

AzureOpenAIProxy.sln

src/AzureOpenAIProxy.PlaygroundApp/Models/AdminEventDetails.cs

src/AzureOpenAIProxy.PlaygroundApp/Models/EventDetails.cs

test/AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsTests.cs

- AzureOpenAIProxy.sln 파일 이전 내용으로 되돌리기 - AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsTests.cs 테스트 코드 삭제

KYJKY · 2024-09-03T15:39:30Z

~~#295 작업 후, 다시 돌아와서 수정하도록 하겠습니다.~~

바로 이어서 진행하도록 하겠습니다!

justinyoo

수정사항 코멘트 남겨뒀습니다!

src/AzureOpenAIProxy.PlaygroundApp/Components/UI/Admin/NewEventDetailsComponent.razor

justinyoo · 2024-09-07T02:47:47Z

@KYJKY 컨플릭 해소해 주세요!

- Id 파라미터 추가 - 버튼 Id 추가 - TextFieldType 추가 - Id 값 kebab-casing 수정

justinyoo

코멘트 남겨뒀습니다.

src/AzureOpenAIProxy.PlaygroundApp/Components/UI/Admin/NewEventDetailsComponent.razor

src/AzureOpenAIProxy.PlaygroundApp/Models/AdminEventDetails.cs

src/AzureOpenAIProxy.PlaygroundApp/Models/EventDetails.cs

src/AzureOpenAIProxy.PlaygroundApp/Components/UI/Admin/NewEventDetailsComponent.razor

- 기본 날짜/시간 설정 추가 - 이벤트 추가/취소 버튼 이벤트 바인딩 추가 - Max Token Cap, Daily Request Cap 값 바인딩

This reverts commit 197d7ee.

KYJKY · 2024-10-01T16:27:24Z

현재 피드백 사항 적용 및 테스트 코드 추가 완료하였습니다!

질문사항

사용자의 브라우저 타임존을 가져오기 위해 추가한 timezone.js 파일이 있는데 해당 경로로 추가 및 명명을 커밋한 상태로 둬도 괜찮을까요?
사용자의 타임존을 가져오는 코드를 다른 컴포넌트에서 사용할 수 있도록 클래스로 빼는 작업이 필요할까요?
[요소검사/타임존초기화/이벤트시작시간/이벤트종료시간] 에 대한 테스트를 추가해둔 상태인데 현재 Task에서 별도로 추가가 필요한 테스트 항목이 있을까요?

justinyoo · 2024-10-02T08:54:02Z

@KYJKY 빌드가 깨집니다

justinyoo · 2024-10-02T08:54:56Z

사용자의 브라우저 타임존을 가져오기 위해 추가한 timezone.js 파일이 있는데 해당 경로로 추가 및 명명을 커밋한 상태로 둬도 괜찮을까요?

브라우저 타임존 대신 시스템 타임존에 의존해야 합니다. timezone.js 파일은 어디서 온 건가요?

justinyoo · 2024-10-02T08:56:08Z

사용자의 타임존을 가져오는 코드를 다른 컴포넌트에서 사용할 수 있도록 클래스로 빼는 작업이 필요할까요?

사용자의 타임존은 이 화면과 이벤트 업데이트하는 화면 말고는 쓸 일이 없습니다.

justinyoo · 2024-10-02T08:57:25Z

[요소검사/타임존초기화/이벤트시작시간/이벤트종료시간] 에 대한 테스트를 추가해둔 상태인데 현재 Task에서 별도로 추가가 필요한 테스트 항목이 있을까요?

충분한 듯 싶습니다!

justinyoo · 2024-10-05T00:08:45Z

@KYJKY 테스트 실패합니다. 날짜 포맷 다시 확인해 보세요!

…nverter

test/AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsPageTests.cs

…nverter

test/AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsPageTests.cs

src/AzureOpenAIProxy.PlaygroundApp/AzureOpenAIProxy.PlaygroundApp.csproj

test/AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsPageTests.cs

- Delete TimeZoneConverter - Split input event datetime test

justinyoo · 2024-10-10T11:23:03Z

@KYJKY 테스트 깨지는 부분 main 브랜치에서 반영해 주세욥

justinyoo

LGTM! 고생하셨습니다!

KYJKY added 3 commits September 1, 2024 18:08

Feat: 어드민 UI 컴포넌트 Page 추가

986d5fe

Test: 어드민 UI 컴포넌트 테스트 추가

aa36c46

Feat: 버튼 추가

b6d13d1

justinyoo assigned KYJKY Sep 1, 2024

justinyoo added ossca admin team-kim task labels Sep 1, 2024

justinyoo linked an issue Sep 1, 2024 that may be closed by this pull request

[Admin] Component: Create event details - UI component #214

Closed

justinyoo requested changes Sep 1, 2024

View reviewed changes

Merge branch 'main' into feature/214-create-event-details

15b36d4

justinyoo requested changes Sep 3, 2024

View reviewed changes

KYJKY added 2 commits September 4, 2024 00:36

Fix: 피드백 적용

765a28f

- AzureOpenAIProxy.sln 파일 이전 내용으로 되돌리기 - AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsTests.cs 테스트 코드 삭제

Update PlaygroundApp Model 최신화

7af315d

justinyoo requested changes Sep 6, 2024

View reviewed changes

Merge: main > Task aliencube#214

560c359

Refactor: 컴포넌트 수정

b578fb2

- Id 파라미터 추가 - 버튼 Id 추가 - TextFieldType 추가 - Id 값 kebab-casing 수정

justinyoo requested changes Sep 7, 2024

View reviewed changes

src/AzureOpenAIProxy.PlaygroundApp/Components/UI/Admin/NewEventDetailsComponent.razor Outdated Show resolved Hide resolved

KYJKY added 7 commits September 10, 2024 00:54

Feat: NodaTime을 사용하여 Time Zone Option 추가

70cbc7c

Feat: 기본값 및 이벤트 설정

59d5eb7

- 기본 날짜/시간 설정 추가 - 이벤트 추가/취소 버튼 이벤트 바인딩 추가 - Max Token Cap, Daily Request Cap 값 바인딩

Refactor: 불필요한 코드 정리

6524e01

Refactor: Time Zone Select 높이 수정

77681cc

Refactor: CSS 적용 방식 수정 - 외부 스타일 시트 적용

197d7ee

Revert "Refactor: CSS 적용 방식 수정 - 외부 스타일 시트 적용"

ecb1b2a

This reverts commit 197d7ee.

Refactor: 이벤트 종료 날짜 기본값 수정 (오늘 기준 다음날로 적용)

0fbcc1f

KYJKY added 2 commits October 2, 2024 01:13

Refactor: NewEventDetailsComponent.razor

60e851a

Test: Add init timezone test

c636124

KYJKY added 3 commits October 4, 2024 00:51

Merge branch 'main' into feature/214-create-event-details

1a3cd8c

Fix: Browser Timezone > System Timezone

6bf9e4f

Fix: Test error fix (Now > UtcNow)

7c13476

KYJKY added 3 commits October 5, 2024 12:29

Fix: add attribute Culture to FluentDatePicker

77fd773

Fix: Add culture info in OnAfterRenderAsync

d5b9d02

Feat: Convert from Windows timezone to IANA timezone using TimeZoneCo…

4587101

…nverter

justinyoo requested changes Oct 5, 2024

View reviewed changes

KYJKY added 3 commits October 5, 2024 14:38

Test: Convert from Windows timezone to IANA timezone using TimeZoneCo…

225ff36

…nverter

Fix: Check OS to get timezone

ad7ee1f

Test: Refactoring NewEventDetailsPageTests

50d38bf

justinyoo requested changes Oct 5, 2024

View reviewed changes

test/AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsPageTests.cs Outdated Show resolved Hide resolved

test/AzureOpenAIProxy.PlaygroundApp.Tests/Pages/NewEventDetailsPageTests.cs Outdated Show resolved Hide resolved

KYJKY added 2 commits October 5, 2024 20:22

Test: Refactoring NewEventDetailsPageTests

02ff26f

Merge branch 'main' into feature/214-create-event-details

7e3bdfa

justinyoo requested changes Oct 6, 2024

View reviewed changes

Refactor: Refactoring NewEventDetailsComponent and test

77cf3a1

- Delete TimeZoneConverter - Split input event datetime test

KYJKY added 3 commits October 12, 2024 11:38

Merge branch 'main' into feature/214-create-event-details

ff6ed0e

Test: Add NewEventDetailsPageTests.cs to AppHost test

8713a32

Merge branch 'main' into feature/214-create-event-details

e77caa0

justinyoo approved these changes Oct 12, 2024

View reviewed changes

justinyoo merged commit 97a3b53 into aliencube:main Oct 12, 2024
1 check passed

sikutisa mentioned this pull request Oct 17, 2024

[Admin] Component: Update event details - UI component (view) #335

Open

2 tasks

[Admin] Component: Create event details - UI component #214 #293

[Admin] Component: Create event details - UI component #214 #293

Uh oh!

Conversation

KYJKY commented Sep 1, 2024

Task

작업사항

궁금한/잘 모르겠는 부분

추후 작업 예정

Uh oh!

justinyoo commented Sep 1, 2024

Uh oh!

justinyoo commented Sep 1, 2024

Uh oh!

justinyoo left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KYJKY commented Sep 3, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

justinyoo left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

justinyoo commented Sep 7, 2024

Uh oh!

justinyoo left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KYJKY commented Oct 1, 2024

질문사항

Uh oh!

justinyoo commented Oct 2, 2024

Uh oh!

justinyoo commented Oct 2, 2024

Uh oh!

justinyoo commented Oct 2, 2024

Uh oh!

justinyoo commented Oct 2, 2024

Uh oh!

justinyoo commented Oct 5, 2024

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

KYJKY commented Sep 3, 2024 •

edited

Loading