[#282] AccountView에서 타 플랫폼 선택을 메뉴가 아닌 Form으로 수정하고 연결된 형태를 표시한다

[opficdev]

• closed AccountView에서 타 플랫폼 선택을 메뉴가 아닌 Form으로 수정하고 연결된 형태를 표시한다 #282

[gemini-code-assist]

Summary of Changes

Hello, I'm Gemini Code Assist¹! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

이 풀 리퀘스트는 AccountView의 사용자 경험을 개선하기 위해 계정 연동 인터페이스를 재구성합니다. 기존의 메뉴 기반 선택 방식 대신, 이제 각 소셜 계정의 연결 상태를 명확하게 보여주고 직접 '연결' 또는 '연결 해제'할 수 있는 폼 형태의 UI를 제공합니다. 이는 사용자가 자신의 계정 연결 상태를 한눈에 파악하고 더 직관적으로 관리할 수 있도록 돕습니다.

Highlights

• 계정 연동 UI 변경: 계정 연동 UI가 메뉴 방식에서 각 소셜 계정의 연결 상태를 직접 표시하고 제어하는 폼 방식으로 변경되었습니다.
• 직관적인 연결/연결 해제 버튼: 각 소셜 계정 옆에 '연결' 또는 '연결 해제' 버튼이 직접 표시되어 사용자 상호작용이 개선되었습니다.
• 코드 리팩토링: 계정 제공자(provider)의 이미지와 이름을 표시하는 로직이 providerContent라는 재사용 가능한 헬퍼 함수로 추출되어 코드 중복이 줄었습니다.
• 로컬라이제이션 업데이트: UI 변경에 맞춰 로컬라이제이션 문자열이 업데이트되었습니다.

Changelog

• DevLog/Resource/Localizable.xcstrings
  • 사용되지 않는 "계정 삭제" 및 "새 계정 연동" 문자열이 제거되었습니다.
  • 새로운 UI에 필요한 "소셜 계정", "연결", "연결 해제" 문자열이 추가되었습니다.
• DevLog/UI/Setting/AccountView.swift
  • 계정 연동 섹션이 툴바 메뉴에서 직접적인 목록 형태로 변경되었습니다.
  • 각 소셜 계정의 연결 상태에 따라 '연결' 또는 '연결 해제' 버튼이 표시되도록 구현되었습니다.
  • 계정 제공자 정보를 표시하는 providerContent 헬퍼 함수가 추가되어 코드 재사용성을 높였습니다.

Activity

• 현재까지 이 풀 리퀘스트에 대한 사람의 활동(댓글, 리뷰 등)은 없습니다.

Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point by creating a comment using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands on the current page.

Feature	Command	Description
Code Review	/gemini review	Performs a code review for the current pull request in its current state.
Pull Request Summary	/gemini summary	Provides a summary of the current pull request in its current state.
Comment	@gemini-code-assist	Responds in comments when explicitly tagged, both in pull request comments and review comments.
Help	/gemini help	Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.

Footnotes

1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩

[gemini-code-assist]

Code Review

안녕하세요. AccountView의 계정 연동 UI를 개선하는 PR 잘 보았습니다. 기존의 메뉴와 스와이프 방식 대신 Form을 사용하여 모든 소셜 계정의 연결 상태를 명확히 보여주고 바로 연결/해제할 수 있도록 변경한 점이 사용자 경험을 크게 향상시킬 것으로 보입니다. 또한, 중복되는 UI 코드를 providerContent 함수로 추출하여 코드 가독성과 유지보수성을 높인 점도 훌륭합니다. 코드 개선을 위해 한 가지 작은 제안을 리뷰 댓글로 남겼으니 확인 부탁드립니다.

[gemini-code-assist]

providerContent 함수 내에서 formattedProviderName(provider)가 두 번 호출되고 있습니다. 불필요한 중복 호출을 피하고 가독성을 높이기 위해, 결과를 한 번만 호출하여 지역 변수에 저장한 뒤 사용하는 것을 권장합니다.

        let name = formattedProviderName(provider)
        Image(name)
            .resizable()
            .scaledToFit()
            .frame(width: UIFont.labelFontSize)
        Text(name)