Skip to content
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

[CAT-207] 온보딩 - 고양이 선택하기 뷰 UI및 기능 #29

Merged
merged 14 commits into from
Aug 16, 2024

Conversation

Jihyun247
Copy link
Collaborator

[CAT-207] 온보딩 - 고양이 선택하기 뷰 UI및 기능

무엇에 관한 PR 인가요? 🙋

온보딩 - 고양이 선택하기 뷰 UI및 기능

어떤 것을 작업하셨나요? 🛠

  • 고양이 목록 서버통신
  • 선택한 고양이 put 서버통신
  • 고양이 구조체 정의
  • usernotification request

🌱 PR Point

  • 고쳐야할 부분이 많은데 우선 빨리 피알 날리고 수정하는게 좋을듯해서 날립니다 러프하게 봐주세욥 ..

@Jihyun247 Jihyun247 added the enhancement New feature or request label Aug 16, 2024
@Jihyun247 Jihyun247 requested a review from devMinseok August 16, 2024 11:29
@Jihyun247 Jihyun247 self-assigned this Aug 16, 2024
Copy link

linear bot commented Aug 16, 2024

Copy link
Member

@devMinseok devMinseok left a comment

Choose a reason for hiding this comment

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

고생했어! rebase해서 develop브랜치 변경사항이랑 싱크맞춰줘!
아 그리고 혹시 모듈만들때 make module안쓰고 있어?

@Jihyun247
Copy link
Collaborator Author

고생했어! rebase해서 develop브랜치 변경사항이랑 싱크맞춰줘! 아 그리고 혹시 모듈만들때 make module안쓰고 있어?

오잉 아니 !! 이번 이슈에서 make module 로 레이어 지정하고, 모듈 이름 지정해서 UserService 만들어줬어 ! 왜염 ?

@devMinseok
Copy link
Member

아아 이거 원래 스캐폴딩할때 파일헤더가 자동으로 붙어야하는데 UserService에 Project.swift보니까 파일 헤더가 없길래 혹시나해서!

devMinseok
devMinseok previously approved these changes Aug 16, 2024
@@ -60,7 +60,7 @@ public struct SplashCore {

extension SplashCore {
private func checkDeviceIDExist() -> Effect<Action> {
let deviceID = keychainClient.read(key: KeychainKeys.deviceID.rawValue) ?? getDeviceUUID()
let deviceID = keychainClient.read(key: "mohanyang_keychain_device_id") ?? getDeviceUUID()
Copy link
Member

Choose a reason for hiding this comment

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

요거도 key 상수로 빼주면 좋을거 같아!

@@ -81,7 +81,7 @@ extension SplashCore {

private func getDeviceUUID() -> String {
guard let uuid = UIDevice.current.identifierForVendor?.uuidString,
keychainClient.create(key: KeychainKeys.deviceID.rawValue, data: uuid) else {
keychainClient.create(key: "mohanyang_keychain_device_id", data: uuid) else {
Copy link
Member

Choose a reason for hiding this comment

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

요거도!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

홀리몰리 gpt다 감삼다 항상 맘이 급하면 한번에 제대로 되는것이없네

@Jihyun247 Jihyun247 merged commit 52f0a32 into develop Aug 16, 2024
1 check passed
@Jihyun247 Jihyun247 deleted the feature/CAT-207 branch August 16, 2024 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants