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

[Frontend] The loading state only last for 5 seconds when reporting #573

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

badukwei
Copy link
Collaborator

@badukwei badukwei commented Nov 5, 2024

Summary

修改 report flow 與更新相關組件

Linked Issue

close #554

Details

  • Remove the reporting success dialog
  • Remove the reporting failure dialog
  • 在載入對話框上新增關閉按鈕
  • 顯示載入對話框 5 秒並自動關閉對話框

Copy link
Collaborator

@Xiawpohr Xiawpohr left a comment

Choose a reason for hiding this comment

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

Thanks for your contribution

{isSuccess && <ReportFormSubmitSuccess onClose={onCloseDialog} />}
<ReportFormSubmitting
isOpen={isSubmitting}
onClose={onCloseDialog}
Copy link
Collaborator

Choose a reason for hiding this comment

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

按下右上角的 close button 時,沒有關閉 ReportFormSubmitting 哦

clearTimeout(timer)
}
}
}, [isPending, onCloseDialog])
Copy link
Collaborator

Choose a reason for hiding this comment

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

如果當前 epoch 已做過 user state transition 時,reportComment 的時間會少於 5 秒,因而導致 5 秒過後不會自動關閉 ReportFormSubmitting。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[UST-244] [Frontend] The loading state only last for 5 seconds when reporting
2 participants