Closed tataihono closed 18 hours ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 89.14%. Comparing base (
cf2f1f2
) to head (40fc88e
). Report is 5 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
The latest updates on your projects.
Name | Status | Preview | Updated (UTC) |
---|---|---|---|
videos-admin | ✅ Ready | videos-admin preview | Thu Oct 17 01:33:00 UTC 2024 |
The latest updates on your projects.
Name | Status | Preview | Updated (UTC) |
---|---|---|---|
watch | ✅ Ready | watch preview | Thu Oct 17 01:34:25 UTC 2024 |
The latest updates on your projects.
Name | Status | Preview | Updated (UTC) |
---|---|---|---|
journeys | ✅ Ready | journeys preview | Thu Oct 17 01:33:21 UTC 2024 |
The latest updates on your projects.
Name | Status | Preview | Updated (UTC) |
---|---|---|---|
journeys-admin | ✅ Ready | journeys-admin preview | Thu Oct 17 01:34:22 UTC 2024 |
This pull request includes changes to the
AnalyticsItem
andResponsesItem
components in theapps/journeys-admin
project. The main focus is on improving the user interface by adding visitor counts to the toolbar items and updating test cases accordingly.Enhancements to Toolbar Items:
apps/journeys-admin/src/components/Editor/Toolbar/Items/Item/Item.tsx
: Addedcount
andcountLabel
properties to theItem
component to display visitor counts. Updated the rendering logic to include these properties foricon-button
variants. [1] [2] [3] [4]Updates to AnalyticsItem:
apps/journeys-admin/src/components/Editor/Toolbar/Items/AnalyticsItem/AnalyticsItem.tsx
: Switched fromuseQuery
touseLazyQuery
for fetching visitor data and addeduseEffect
to load data when the journey ID changes. Updated the component to use the newcount
andcountLabel
properties. [1] [2] [3]apps/journeys-admin/src/components/Editor/Toolbar/Items/AnalyticsItem/AnalyticsItem.spec.tsx
: Updated test cases to reflect changes in theAnalyticsItem
component, including the new visitor count display and theicon-button
variant. [1] [2]Updates to ResponsesItem:
apps/journeys-admin/src/components/Editor/Toolbar/Items/ResponsesItem/ResponsesItem.tsx
: Added avariant
prop to allow different button styles and updated the component to use the newcount
andcountLabel
properties. [1] [2] [3] [4]apps/journeys-admin/src/components/Editor/Toolbar/Items/ResponsesItem/ResponsesItem.spec.tsx
: Updated test cases to reflect changes in theResponsesItem
component, including the new visitor count display and theicon-button
variant. [1] [2]Storybook Updates:
apps/journeys-admin/src/components/Editor/Toolbar/Items/Item/Item.stories.tsx
: Added a new story forIconButtonWithCountItem
to demonstrate the newcount
andcountLabel
properties.apps/journeys-admin/src/components/Editor/Toolbar/Items/ResponsesItem/ResponsesItem.stories.tsx
: Updated the story to use theicon-button
variant.These changes enhance the user interface by providing more informative toolbar items and ensuring the components are well-tested.