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

Feat/#497 오픈소스 라이선스 표기 #502

Open
wants to merge 4 commits into
base: dev
Choose a base branch
from

Conversation

sujin9
Copy link
Collaborator

@sujin9 sujin9 commented Oct 17, 2023

🔥 연관 이슈

📝 작업 요약

마이페이지 > 오픈소스 라이선스에서 오픈소스 라이브러리 라이선스를 볼 수 있어요.

🔎 작업 상세 설명

oss-licenses-plugin을 이용했어요.
OssLicensesMenuActivity에서 오픈소스 라이브러리 리스트 및 라이선스를 확인할 수 있습니다.
공식문서: https://developers.google.com/android/guides/opensource?hl=en

🌟 리뷰 요구 사항

코드 변화는 거의 없어 simple 하지만, debug에서는 안보이고 release여야만 확인할 수 있어서 a little bit 귀찮을 수 있어요.
이 부분은 최하단에 스크린샷을 첨부하겠습니다.
그리고, 마이페이지에서 이것저것 하단 메뉴가 많아지는데 정렬 순서 괜찮은지 봐주시면 감사하겠습니다 ~

@sujin9 sujin9 self-assigned this Oct 17, 2023
@sujin9 sujin9 added the 💚 Android for Android label Oct 17, 2023
@github-actions
Copy link

github-actions bot commented Oct 17, 2023

Test Results

43 tests  ±0   43 ✔️ ±0   0s ⏱️ ±0s
  8 suites ±0     0 💤 ±0 
  8 files   ±0     0 ±0 

Results for commit 3e77f2d. ± Comparison against base commit 55cc238.

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@Choisehyeon Choisehyeon left a comment

Choose a reason for hiding this comment

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

문의하기랑 오픈소스 라이센스 버튼 위치 괜찮은 것 같아요!!!
오픈 소스 표기는 저도 처음 경험해보는데 공식문서에 잘 나와있네요👍👍
release 버전을 확인할 수 없어서 오픈소스를 열어볼 수가 없지만 아까 하티 폰으로 연락했으니 approve 하겠습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💚 Android for Android
Projects
None yet
Development

Successfully merging this pull request may close these issues.

오픈소스 표기
2 participants