-
Notifications
You must be signed in to change notification settings - Fork 167
Content Insights App [MAPS-118] #10505
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
base: master
Are you sure you want to change the base?
Conversation
* initial structure * removing es lint flag * adding images sections * adding tests + refactors * modifying ranges and styles * modifying validation checks * modifying images styles * removing unused const * changing description color * regenerating package lock * addressing some pr comments * refactoring error logic * removing unused import
* First version fetching entries with TanStack Query * adding error and loading state and formatting * Adding page location and entries fetch tests * fixing lint errors * removing unused library * Addressing PR comments * reducing batch size to 15 * changing comments from tests * solving conflicts with feature branch * lint fixes * adding missing import while resolving conflicts * adding node to the tsconfig types * addressing last pr comments
* fetching actions * renaming for consistency * rename * Updating to include scheduling
* adding metrics calculator with first metric case * adding average time to publish metric * adding scheduled metric * adding recently published and needs update metrics * adding installation parameters usage * adding initial test for components = structure refactor * adding metrics calculator tests * changes in card styles * removing mocked entries and scheduled actions * mini refactors * resolving conflicts after merging another PR * adding Home tests + removing refresh button * Removing unused refetch * connecting the scheduled actions * fixing es-lint error * prettier fix * removing status check * using setDate * removing metrics file and merging it to MetricsCalculator * adding date calculator and refactors in metrics calculator * refactor in DateCalculator and MetricsCalculator * refactors in MetricsCalculator and useScheduledActions
* first version of maps-190 * correction on metric skeleton * adding and correcting tests
* Only the hook * Readding prettier * Using select to avoid fetching too many times * Reusing fetch and using aliases EVERYWHERE * Removing unnecesary filters * Lint and making sdk more generic * fix test * Fix build issue * fix prettier
* Dividing PR * lint fixes * Fix for dashboard * Runing prettier * Tests fixing * Lint * More fixes * more fixes * minor rebase fixes * Removing unnecesary truncation
* Dividing PR * Runing prettier * Tests fixing * Lint * More fixes * adding modal * Add styles for Dashboard and ReleasesTable components; implement UnscheduleReleaseModal for managing scheduled releases * Testing table * Autocomples einstead of selects and * Separating concerns * Readding tests * Fix a few things post rebase * Prettier * Removing unnecesary logic * Fix prettier * fix lint * refetchReleases * removing form * Update apps/content-production-dashboard/src/utils/Validator.ts Co-authored-by: Franco Banfi <[email protected]> * refactoring a bit * Update apps/content-production-dashboard/src/components/RescheduleModal.tsx Co-authored-by: Franco Banfi <[email protected]> * fixing rename * tests * prettier --------- Co-authored-by: Franco Banfi <[email protected]>
…[MAPS-124] (#10452) * adding hooks required for the content publishing trends * adding tests for useContentTypes hook
…ucture [MAPS-124] (#10453) * adding hooks required for the content publishing trends * adding tests for useContentTypes hook * wip: base structure for the Content Publishing Trends * adding code to generate the content type tab line by id but legends by name * adding connection between config screen and trends chart * adding tests and refactors * refactors in tracked content types * adding dependency and parameter to useMemo * adding installation parameters hook * fixing test * fixing order of async refetch * removing unused method * adding mini refactors * removing chart custom tooltip and moving colors to a style file * addressing first comments of the PR * renaming files + moving code to different files + refactors
* Main UI * Making empty state better * Pluralizing tabs * Empty state better * Lint * Prettier violation fix * fix test * no title * lint
* adding hooks required for the content publishing trends * adding tests for useContentTypes hook * wip: base structure for the Content Publishing Trends * adding code to generate the content type tab line by id but legends by name * adding connection between config screen and trends chart * adding tests and refactors * refactors in tracked content types * adding dependency and parameter to useMemo * adding installation parameters hook * fixing test * fixing order of async refetch * removing unused method * adding mini refactors * removing chart custom tooltip and moving colors to a style file * addressing first comments of the PR * renaming files + moving code to different files + refactors * refactoring trendsDataProcessor + tests * refactoring props and chart line generation * removing unused prop * changing fors for forEachs * adding average time to publish logic and refactors
* adding hooks required for the content publishing trends * adding tests for useContentTypes hook * wip: base structure for the Content Publishing Trends * adding code to generate the content type tab line by id but legends by name * adding connection between config screen and trends chart * adding tests and refactors * refactors in tracked content types * adding dependency and parameter to useMemo * adding installation parameters hook * fixing test * fixing order of async refetch * removing unused method * adding mini refactors * removing chart custom tooltip and moving colors to a style file * addressing first comments of the PR * renaming files + moving code to different files + refactors * refactoring trendsDataProcessor + tests * refactoring props and chart line generation * removing unused prop * changing fors for forEachs * adding average time to publish logic and refactors * adding content types autocomplete * fixing tests * adding tests and fixing warnings * fixing content trends tab test * fixing config screen test * fix in rebase * formatting with prettier * moving useContentTypes hook from ContentTrendsTabs to Dashboard
* adding hooks required for the content publishing trends * adding tests for useContentTypes hook * wip: base structure for the Content Publishing Trends * adding code to generate the content type tab line by id but legends by name * adding connection between config screen and trends chart * adding tests and refactors * refactors in tracked content types * adding dependency and parameter to useMemo * adding installation parameters hook * fixing test * fixing order of async refetch * removing unused method * adding mini refactors * removing chart custom tooltip and moving colors to a style file * addressing first comments of the PR * renaming files + moving code to different files + refactors * refactoring trendsDataProcessor + tests * refactoring props and chart line generation * removing unused prop * changing fors for forEachs * adding average time to publish logic and refactors * adding content types autocomplete * fixing tests * adding tests and fixing warnings * fixing content trends tab test * fixing config screen test * fix in rebase * adding content types autocomplete * changing height * adding tooltip and tests * formatting with prettier * formatting with prettier * removing unnecessary default value fallback
* Main UI * Pluralizing tabs * Lint * moving to utils as we need it for tables * base table * wip hook * prettier * adding usememos everywhere! * tests * Improving wrapper * fix * Adding timezone for scheduled for * Making stuff easier * removing this * removing this as well * more cleanup * simplifying some more * fixing test * fix: scheduled content table spec * Fixing comments * Fix merge
* Main UI * base table * wip hook * prettier * adding usememos everywhere! * tests * Improving wrapper * Making stuff easier * removing this * removing this as well * more cleanup * wip * Refactor RecentlyPublishedTable to include error handling and reorder table columns for better clarity. Update useRecentlyPublishedContent hook to return error state and utilize utility functions for unique IDs extraction. * fix and refactor * Adding test and * commit * tests for table * fixing post merge * fixing comments
* base table * Update content types usage * sort by age * Use const for calculating age * Including content types * refactoring tables (#10484) * fix and refactoring tests * hook tests * fix
* Changes to config screen * fix skeleton * Creator view options * Fixing users fetching * wip * fix styles * Sorting for real * fixing tests * more test fixing * removing pills * no initial creators * remove unused icon * updating paragraph * Remove filter by content type for creator data fetchin * fixing pill issue * Config screen tooltip and divider for tabs * Refactor error handling in ContentTable and optimize ContentTypeMultiSelect logic
* Renaming app * Update ConfigScreen terminology from "Content Production Dashboard" to "Content Insights" for consistency across the application. * Changing appearance to update naming
* Refactor Empty State Handling in Content Tables * fix signature issue * prettier
Wiz Scan Summary
To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension. |
* refactoring a bit * Refactoring utils * fix const
…for Contentful uploads
| }); | ||
| }, [filteredEntries, contentTypes, usersMap, defaultLocale]); | ||
|
|
||
| const skip = page * ITEMS_PER_PAGE; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This value is used multiple places, could just be a constant
| To make this dashboard your default home page, select the gear icon in the top right | ||
| corner of your Contentful navigation. | ||
| </Paragraph> | ||
| <Image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Consider using SVGs so that it maintains the proper scaling values rather than a hardcoded image size
| <Paragraph>Select "Content Insights" and click save.</Paragraph> | ||
| <Image | ||
| alt="An image showing Contentful Home location appearance settings" | ||
| height="257px" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same comment about scaling image values without losing quality. Maybe this isn't so easy and is pretty minor so up to you to decide what to do
apps/content-insights/src/components/ContentTypeMultiSelect.tsx
Outdated
Show resolved
Hide resolved
| return result; | ||
| } | ||
|
|
||
| export function subMonths(base: Date, months: number): Date { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ryunsong-contentful
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall good work and well laid out.
Three overall things that my comments cover
- A few UI/UX questions
- Just small/minor code fixes
- Potential bug scenarios
ryunsong-contentful
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry meant as a comment not and approve
…improved content type retrieval



Purpose
Release of the Content Insights App. Dashboard that shows insights about the content the user has in contentful.
Approach
For Page and optionally Home locations, these are the features the dashboard part has:
Configuration screen:
Testing steps
Grabacion.de.pantalla.2026-01-30.a.las.10.16.58.a.m.mov