feat(feedback): frontend to display summary #93567
Conversation
…d uses create_feedback
…sh/cache-ai-summary
…sh/cache-ai-summary merge the branch
…mmary-frontend merge
|
Jesse mentioned he'll get a design out for this with updated states for loading and if there aren't enough feedbacks. Should we merge and experiment with this for now, and make those changes in a follow-up PR? Or wait to incorporate those changes here. |
i think it's fine to merge what we're happy with here & make changes in a followup PR |
| const {selection} = usePageFilters(); | ||
|
|
||
| const normalizedDateRange = normalizeDateTimeParams(selection.datetime); |
There was a problem hiding this comment.
I've updated the way the date range filtering is done and from my testing (and looking at the code) this seems to work. Can you please have a look @ryan953 @michellewzhang?
There was a problem hiding this comment.
woo, fewer weird things going on!
|
hmm, maybe the error you're seeing is similar to the one Ryan was seeing? like the LLM not generating in the right format. I'm investigating now Edit: looking at Sentry, it does seem that you were facing the same parsing error that Ryan encountered (I am encountering this too) |
vishnupsatish
removed
the
Scope: Backend
…mmary-frontend merge
Linear ticket: https://linear.app/getsentry/issue/REPLAY-294/frontend-to-display-the-summary
Loading (placeholder) state:
Summary:
Error state:
To be honest I don't know how to implement the "Read more" logic that's on the Figma, image below:
I think I might be able to add the "Read more" on the next line, but I'm wondering that if the response is meant to be only a few sentences, do we need it?
--