Skip to content

[Feat] #21 - 공통 토스트 구현 #70

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.

Sign up for GitHub

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

Jump to bottom
Merged
hyowon612 merged 1 commit into from
Jan 17, 2024
Merged

[Feat] #21 - 공통 토스트 구현 #70

hyowon612 merged 1 commit into from
Jan 17, 2024

Conversation

@hyowon612
Copy link
Collaborator

@hyowon612 hyowon612 commented Jan 17, 2024 •
edited
Loading

⭐️Issue

#21

🌟Motivation

공통 토스트 구현했습니다.

🌟Key Changes

  • 토스트 케이스 enum으로 정의했습니다.
  • 토스트를 띄우는 메서드를 UIViewController+ 에 정의했습니다.
  • 토스트를 띄우고 싶은 시점에서 아래 메서드를 호출하면 됩니다.
showToast(.memoDelete)

🌟Simulation


🌟To Reviewer


🌟Reference

@hyowon612 hyowon612 self-assigned this Jan 17, 2024
ena-isme
ena-isme approved these changes Jan 17, 2024
View reviewed changes
Copy link
Member

@ena-isme ena-isme left a comment •
edited
Loading

Choose a reason for hiding this comment

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

스크린샷 2024-01-17 오후 1 05 17

굳이에요 굳

WSSiOS/WSSiOS/Source/Presentation/Base/WSSToastView.swift
}
}
}
}
Copy link
Member

@ena-isme ena-isme Jan 17, 2024

Choose a reason for hiding this comment

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

W3
번거로웠을텐데 수고많았숩니둥 냠냠밍

@hyowon612 hyowon612 merged commit 10db2b3 into main Jan 17, 2024
@hyowon612 hyowon612 deleted the Feat/#21 branch January 17, 2024 04:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ena-isme ena-isme ena-isme approved these changes

@Naknakk Naknakk Awaiting requested review from Naknakk

@Guryss Guryss Awaiting requested review from Guryss

Assignees

@hyowon612 hyowon612

Labels

haku veryGoodDeveloper

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@hyowon612 @ena-isme @Guryss