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

[Dashboard] Should show cached values from the previous query when error raised at @bento/core #3

Open
junhoyeo opened this issue May 18, 2022 · 1 comment
Assignees

Comments

@junhoyeo
Copy link
Member

  • 지금은 에러 발생했다는 것을 표시하지 않음
    • 디버깅 용으로 에러 발생 위치(실행하려 했던 메소드), 에러 메시지를 반환해줄 필요가 있음
  • 클라에서 이전 값 로컬스토리지 등에 저장하다가 표시/업데이트
@junhoyeo junhoyeo changed the title @dashboard/core 에서 에러가 발생하면 클라이언트에서 이전에 캐싱된 값을 사용하도록 함 @dashboard/core 에서 에러가 발생하면 클라이언트에서 이전에 캐싱된 값을 사용하기 May 18, 2022
@junhoyeo junhoyeo changed the title @dashboard/core 에서 에러가 발생하면 클라이언트에서 이전에 캐싱된 값을 사용하기 @bento/core 에서 에러가 발생하면 클라이언트에서 이전에 캐싱된 값을 사용하기 May 26, 2022
@junhoyeo
Copy link
Member Author

스크린샷 2022-05-27 오전 12 06 33

  • 지금은 API 에서 에러가 발생하면 .catch(() => 0) 을 사용해서 기본값을 반환하고 있습니다.
  • 다음과 같이 개선할 수 있습니다.
    • @bento/core 를 외부 API 또는 Serverless Functions 에서 호출하는 경우:
      1. 에러가 발생했음을 클라이언트가 알 수 있게 합니다.
      2. 클라이언트가 에러가 발생했을 때, 마지막으로 쿼리한 값을 사용하거나 기본값을 사용하도록 합니다.
    • @bento/core 를 클라이언트에서 호출하는 경우:
      1. 해당 로직의 호출을 다루는 클래스에 특정 형태의 storage(예: localStorage)를 주입할 수 있게 합니다.
      2. 클라이언트가 에러가 발생했을 때, (1에서 주입한 storage에 저장된) 마지막으로 쿼리한 값을 사용하거나 기본값을 사용하도록 합니다.

@junhoyeo junhoyeo changed the title @bento/core 에서 에러가 발생하면 클라이언트에서 이전에 캐싱된 값을 사용하기 @bento/core 를 사용하는 로직에서, 에러가 발생하면 클라이언트에서 이전에 쿼리했을 때의 값을 사용하기 May 26, 2022
@junhoyeo junhoyeo self-assigned this May 29, 2022
@junhoyeo junhoyeo moved this to Todo in 🚀 Bento MVP Jun 10, 2022
@junhoyeo junhoyeo changed the title @bento/core 를 사용하는 로직에서, 에러가 발생하면 클라이언트에서 이전에 쿼리했을 때의 값을 사용하기 Client should show cached values from the previous query when error raised at @bento/core Jun 23, 2022
@junhoyeo junhoyeo changed the title Client should show cached values from the previous query when error raised at @bento/core [Dashboard] Should show cached values from the previous query when error raised at @bento/core Oct 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Todo
Development

No branches or pull requests

1 participant