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.

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

[ALL] frontend CI main 브랜치 merge 시에도 동작하도록 수정해요 :) #379

Closed
1 task done
Largopie opened this issue Sep 27, 2024 · 0 comments · Fixed by #380
Closed
1 task done

[ALL] frontend CI main 브랜치 merge 시에도 동작하도록 수정해요 :) #379

Largopie opened this issue Sep 27, 2024 · 0 comments · Fixed by #380
Assignees
@Largopie
Labels
⚙️ 환경설정 환경을 설정해요 :) 🐈 프론트엔드 프론트엔드 관련 이슈에요 :)
Milestone
5차 데모데이

Comments

@Largopie
Copy link
Contributor

Largopie commented Sep 27, 2024 •
edited
Loading

이슈 요약

lighthouse에서 성능 측정 시, 아래와 같은 오류가 발생해 카카오 SDK 스크립트를 비동기로 실행시키도록 수정합니다.

image

기존에 카카오 SDK도 async 속성을 추가해서 수정하려했지만, 비동기로 실행됨에 따라서 카카오 SDK 관련 로직이 존재하는 코드가 먼저 실행될 가능성이 있어 적용하지 않기로 했습니다.

상세 작업 내용

~~ - [ ] 카카오 SDK 스크립트를 비동기로 실행하도록 수정~~

  • frontend CI 동작 브랜치 main 추가

레퍼런스
@Largopie Largopie added 🐈 프론트엔드 프론트엔드 관련 이슈에요 :) ♻️ 리팩터링 코드를 깎아요 :) labels Sep 27, 2024
@Largopie Largopie added this to the 5차 데모데이 milestone Sep 27, 2024
@Largopie Largopie self-assigned this Sep 27, 2024
@Largopie Largopie mentioned this issue Sep 27, 2024
Merged
2 tasks
@Largopie Largopie added ⚙️ 환경설정 환경을 설정해요 :) and removed ♻️ 리팩터링 코드를 깎아요 :) labels Sep 27, 2024
@Largopie Largopie changed the title [FE] 카카오 SDK 스크립트를 비동기로 실행해요 :) [ALL] frontend CI main 브랜치 merge 시에도 동작하도록 수정해요 :) Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Largopie Largopie

Labels
⚙️ 환경설정 환경을 설정해요 :) 🐈 프론트엔드 프론트엔드 관련 이슈에요 :)
Projects
2024-momo
Status: Done
Milestone
5차 데모데이
Development

Successfully merging a pull request may close this issue.

[ALL] frontend CI 수정 woowacourse-teams/2024-momo
1 participant
@Largopie