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

플러터 젠 추가 #54

Merged
merged 7 commits into from
Jul 21, 2024
Merged

Conversation

jwson-automation
Copy link
Owner

@jwson-automation jwson-automation commented Jul 20, 2024

설명

플러터 젠은 이미지 파일이나 폰트를 불러올때 큰따옴표 사이에 텍스트 " assets/image1.png " 가 아닌 Assets.Image1 같은 형식으로 텍스트를 지정함으로써 발생 가능한 이슈를 막아주는 패키지 입니다.

https://pub.dev/packages/flutter_gen

변경 사항

  • 플러터 젠 추가

Before : imagePath: 'assets/login_page_images/apple.png'

After : imagePath: Assets.loginPageImages.google.path

체크리스트

< 체크리스트 항목을 확인해 주세요. >

  • 오늘도 행복하게 코딩했는가?
  • release 브랜치를 제대로 최신화 하고 이 브랜치에 merge 했는가?
  • PR 제목은 명확하고 간결한가?
  • PR 에는 하나의 작업에 대한 내용만 포함되었는가?
  • 파일명은 누구나 이해할 수 있게 작성되었는가?
  • camelCase를 사용하였는가? (ThisIsCamelCase, this_is_not_camel_case)
  • 텍스트는 AppStrings.dart 파일에서 가져오고 있는가?
  • 디스코드 채팅방에 완료된 태스크를 알렸는가?

스크린샷

< 여기에 이미지/동영상을 드래그 앤 드랍하면 링크가 생성됩니다. >
sample

참조

관련된 이슈나 PR이 있다면 링크를 추가해주세요.

Copy link
Collaborator

@hongmono hongmono left a comment

Choose a reason for hiding this comment

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

LGTM

@jwson-automation jwson-automation merged commit a094ef3 into release/2024-summer Jul 21, 2024
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants