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

Puppeteer로 성능 측정 스크립트 작성, 중복입장 개발환경에서 가능하도록 수정 #2

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

23YUJIN
Copy link
Collaborator

@23YUJIN 23YUJIN commented Jan 17, 2025

✅ 주요 작업

  • 프로덕션 환경에서만 막아야 했으나, 중복 입장 막기가 작동하는지 확인하기 위해 개발 환경에서도 막히도록 작성되어 있었음. || true 를 제거해 프로덕션 환경에서만 막히도록 수정
    {988C69EE-FCF9-41CA-B6F4-18ABD8D1AFD0}
  • 불편했던 성능 측정 프로세스를 Puppeteer로 자동화하는 스크립트 작성

Puppeteer 성능 측정 캡쳐_그림자

측정이 끝나면 결과는 puppeteer/results 디렉토리에 저장됨.

📚 학습 키워드

git remote

  • 이번 PR과는 무관하지만 첫 PR을 올리기 전 fork한 레포지토리를 remote 저장소로 추가한 뒤 기존 저장소로 push하는 실수를 방지하기 위한 방법을 고민하며 학습. ☔ git 리모트 저장소 push 실수 방지하기 에 해당 내용 작성

Puppeteer

💭 고민과 해결과정

Puppeteer 스크립트 작성 중 비동기 문제를 마주침, 문서 작성 후 링크 추가 예정

📌 이슈 사항

  • 현재 중복 입장 관련해, 새로고침 후 랜딩 페이지로 돌아올 때 클린업 코드가 작동하지 않아 중복 입장 처리가 제대로 안되는 버그가 존재합니다.

@23YUJIN 23YUJIN added the bug Something isn't working label Jan 17, 2025
@23YUJIN 23YUJIN self-assigned this Jan 17, 2025
@23YUJIN 23YUJIN changed the title fix: 🐛 프로덕션 환경에서만 중복 입장을 막도록 || true 제거 Puppeteer로 성능 측정 스크립트 작성, 중복입장 개발환경에서 가능하도록 수정 Jan 17, 2025
@23YUJIN 23YUJIN added the enhancement New feature or request label Jan 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant