Skip to content

feat: Timeline view for company work map / projects#4286

Merged
markoa merged 3 commits into
mainfrom
work-map-timeline
Mar 26, 2026
Merged

feat: Timeline view for company work map / projects#4286
markoa merged 3 commits into
mainfrom
work-map-timeline

Conversation

@markoa
Copy link
Copy Markdown
Member

@markoa markoa commented Mar 26, 2026

No description provided.

Signed-off-by: Marko Anastasov <marko@operately.com>
Copy link
Copy Markdown
Contributor

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Hey - I've left some high level feedback:

  • The WorkMapTimeline.tsx file is quite large and mixes layout, rendering, and date/scale utilities; consider extracting the pure date/column helpers (e.g. calculateRange, buildColumns, startOfWeek/endOfWeek, etc.) into a separate utility module and/or breaking out subcomponents to keep the main component easier to maintain.
  • All date formatting in the timeline is hard-coded to the en-US locale; if the rest of the app is localized, it would be better to use a shared locale setting or formatter helpers instead of inline new Intl.DateTimeFormat("en-US", ...) instances.
Prompt for AI Agents
Please address the comments from this code review:

## Overall Comments
- The `WorkMapTimeline.tsx` file is quite large and mixes layout, rendering, and date/scale utilities; consider extracting the pure date/column helpers (e.g. `calculateRange`, `buildColumns`, `startOfWeek`/`endOfWeek`, etc.) into a separate utility module and/or breaking out subcomponents to keep the main component easier to maintain.
- All date formatting in the timeline is hard-coded to the `en-US` locale; if the rest of the app is localized, it would be better to use a shared locale setting or formatter helpers instead of inline `new Intl.DateTimeFormat("en-US", ...)` instances.

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

markoa added 2 commits March 26, 2026 17:22
Signed-off-by: Marko Anastasov <marko@operately.com>
Signed-off-by: Marko Anastasov <marko@operately.com>
@markoa markoa changed the title feat: Prototype timeline view for company work map / projects feat: Timeline view for company work map / projects Mar 26, 2026
@markoa markoa merged commit cadd16e into main Mar 26, 2026
4 checks passed
@markoa markoa deleted the work-map-timeline branch March 26, 2026 16:52
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.

1 participant