Skip to content

Conversation

HeoJiye
Copy link
Collaborator

@HeoJiye HeoJiye commented Feb 21, 2024

🔮 resolved #551

Note

실시간 쪽 피드백은 어느 타이밍에 띄워야 할까? -> 의논하고 적용 필요

변경 사항

  • TOLD 다시 적용하기

고민과 해결 과정

1️⃣ useTOLD -> useUserFeedback 훅 네이밍 변경

저번에 멘토님이 지적해주셨던 것 반영

  • TOLD 대신에 다른 걸 쓰게 될 수도 있고 TOLD가 이름을 바꿀 수도 있기 때문에 용도에 맞게 네이밍하는 것이 좋다

2️⃣ TOLD 창 띄우는 방식 수정

  • custom event 설정을 시도했으나, 또 실패함.. 업데이트된 문서를 따라서 GTM도 설정하고 했는데도 잘 안됐음..
  • 그래서 야매로 button 방식으로 하되, button를 임시로 생성했다가 클릭하고 없애는 형식으로 함.
  displayForm: (type: string) => {
    const button = document.createElement('button');

    button.innerText = type;
    button.style.display = 'none';

    document.body.appendChild(button);

    button.click();

    document.body.removeChild(button);
  },

(선택) 테스트 결과

  • 배포했을 때 제대로 되는지 체크해야 됨.

@@ -9,3 +9,7 @@ export const ERROR_MESSAGE = {
export const POPUP_MESSAGE = {
SPREAD_TAROT_CARD: '상담자에게 타로 카드가 펼쳐집니다.',
};

export const CHAT_MESSAGE = {
ASK_FEEDBACK: '타로 상담이 끝났어. 마법의 소라고둥에 대해 피드백을 남겨줄래?',
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍👍

Copy link

cloudflare-workers-and-pages bot commented Feb 21, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: cc10681
Status: ✅  Deploy successful!
Preview URL: https://225d1fba.web09-magicconch.pages.dev
Branch Preview URL: https://fe-feature--551-told.web09-magicconch.pages.dev

View logs

@HeoJiye HeoJiye force-pushed the FE/feature/#551-TOLD-다시-적용하기 branch from 43f7f36 to cc10681 Compare February 21, 2024 08:38
@Doosies
Copy link
Collaborator

Doosies commented Feb 21, 2024

👍 👍 👍

@HeoJiye HeoJiye merged commit 00574b7 into dev Feb 21, 2024
@HeoJiye HeoJiye deleted the FE/feature/#551-TOLD-다시-적용하기 branch February 21, 2024 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✅ FEATURE: TOLD 다시 적용하기
3 participants