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

이메일 보내기 기능 #96

Merged
merged 6 commits into from
Mar 11, 2024
Merged

이메일 보내기 기능 #96

merged 6 commits into from
Mar 11, 2024

Conversation

WhiteHyun
Copy link
Member

@WhiteHyun WhiteHyun commented Mar 10, 2024

Screenshots 📸

이메일을 보낼 수 없을 때 보낼 수 있을 때
image IMG_3A39496BC626-1 1



고민, 과정, 근거 💬

편행 1.0 기능을 답습했습니다.
특히 MailView를 구현하는 과정에서 따로 SwiftUI.View를 만들어 사용할 수 있는 방법이 아직 없어서, UIViewControllerRepresentable를 사용했습니다.

...

슬슬 각 공지사항의 행마다 하나의 뷰를 만들어줘야겠다는 생각이 듭니다.
점점 SettingsView가 읽기 어려워지고 있네요.
그리고 객체 주입방법도 어떻게 해야하는 게 좋을지 고민이 많이 됩니다 ㅎㅎ..




- Implemented `MailSheetView` to present a modal view for sending emails upon button press
- Implemented the functionality to send emails directly from the app. If unable to send an email, added code to redirect users to the Mail app store page
- Updated the app to hide images from Accessibility tools to avoid unnecessary distractions.
- Added descriptive text for version information
@WhiteHyun WhiteHyun added 🔧 Enhancement Improvement or Feedback Implementation ⚙️ Settings Settings View labels Mar 10, 2024
@WhiteHyun WhiteHyun added this to the v2.0.0 milestone Mar 10, 2024
@WhiteHyun WhiteHyun requested a review from a team March 10, 2024 07:37
@WhiteHyun WhiteHyun self-assigned this Mar 10, 2024
- Created DeviceProvider protocol to abstract device-related information for use in composing email bodies.
- Modified MailView to rely on DeviceProvider for dynamically populating the mail body with relevant device information, enhancing the detail and relevance of email communications.
@WhiteHyun WhiteHyun marked this pull request as ready for review March 10, 2024 08:27
Copy link
Contributor

@eung7 eung7 left a comment

Choose a reason for hiding this comment

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

👍 수고하셨습니다!

@eung7 eung7 merged commit dec3498 into main Mar 11, 2024
2 checks passed
@eung7 eung7 deleted the feature/settings/89 branch March 11, 2024 02:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔧 Enhancement Improvement or Feedback Implementation ⚙️ Settings Settings View
Projects
Development

Successfully merging this pull request may close these issues.

불만을 제기하기 위해 문의하기를 눌러 이메일로 문의한다.
2 participants