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

[API] cancel #17

Open
amorfati0310 opened this issue Oct 29, 2020 · 0 comments
Open

[API] cancel #17

amorfati0310 opened this issue Oct 29, 2020 · 0 comments

Comments

@amorfati0310
Copy link
Owner

기존에 post로 알림 신청을 날린 부분
취소가 있는게 자연스럽지 않은가 ? -> 이것도 나중에야 생각 났다. 기획단에서 이런 부분을 같이 얘기 미리 해보면 좋았을 텐데
다시 얘기해보니까 -> 알림 신청 취소를 되도록 노출 안 하는 의도가 있었다. 사용자 입장에서는 조금 아쉬울 수는 있어도

  • 만약 취소를 한다면 기존 알림 플로우로 체크하지 않고 바로 신청, 취소 이런 식으로 갈 것 같은데
    어쨌든 이거는 TMI였고

api를 어떻게 주는게 편하세요 질문이 들어왔었다.
/api/.../cancel ? 이 좋을까?
/api/../ 기존것에 Parameter 값으로 true, false로 체크하는게 좋을까?

상황에 따라 다르겠지만 cancel 이 더 의도도 명확하게 드러나고, 편해보였다.

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

No branches or pull requests

1 participant