Skip to content

Conversation

@cannarocks
Copy link
Member

No description provided.

d-beezee and others added 30 commits August 31, 2023 11:19
🔀 refactor(Sentiment.tsx): change secondRowHeight from '465px' to '550px'
🔀 refactor(Sentiment.tsx): rename SentimentChart component to Chart in FlipCard component
🔥 refactor(SentimentList/index.tsx): simplify conditional rendering of loading and no data messages
🔥 refactor(SentimentList/index.tsx): remove unnecessary Row and Col components
🔥 refactor(SentimentList/index.tsx): simplify List component structure
…entiments

The useSentiments hook is added to fetch campaign sentiments based on the provided cid and isPreview flag. It uses the useGetCampaignsByCidUxQuery from the api feature to make the API call.

If the data is still loading or fetching, the hook returns an object with empty sentiments, isLoading set to true, and isError set to false.

If there is no data or sentiment in the response or if there is an error, the hook returns an object with empty sentiments, isLoading set to false, and isError set to true.

Otherwise, the hook maps the sentiment data to an array of objects with title, sentiment, and note properties and returns it along with isLoading set to false and isError set to false.
🐛 fix(SentimentChart.tsx): fix missing import and unused variables

✨ feat(SentimentChart.tsx): add support for displaying sentiment chart and tooltips

🔧 chore(SentimentChart.tsx): update styles and layout of SentimentChart component
marcbon and others added 13 commits September 5, 2023 16:31
🐛 fix(FlipCardContext.tsx): fix breakpoint value to use lg instead of md
🔥 refactor(Overview/index.tsx): remove unused import
…entimentItem

✨ feat(Chart.tsx): add support for loading state and error handling in Chart component
🔧 chore(Chart.tsx): refactor Chart component to improve readability and organization
Co-authored-by: Marco <minucci9292@gmail.com>
✨ feat(Chart.tsx): adjust width of chart based on number of sentiments
✨ feat(Chart.tsx): add legend card to display sentiment information
@cannarocks cannarocks merged commit 0686e27 into master Sep 5, 2023
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

Successfully merging this pull request may close these issues.

4 participants