Skip to content
This repository was archived by the owner on Sep 3, 2025. It is now read-only.

Conversation

@aaronherman
Copy link
Contributor

This pull request introduces a new function, getEventDescription, to dynamically generate event descriptions in the TimelineTab.vue file. This change adds flexibility by allowing special handling of specific events, such as "Case created," based on additional event details like visibility.

Enhancements to event description handling:

  • src/dispatch/static/dispatch/src/case/TimelineTab.vue: Added the getEventDescription function to replace the static descriptionMap lookup. This function includes special handling for the "Case created" event, dynamically incorporating visibility details (e.g., "an open case" or "a restricted case").

@aaronherman aaronherman requested review from Copilot and whitdog47 June 24, 2025 14:05
@aaronherman aaronherman added the enhancement New feature or request label Jun 24, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces the dynamic event description generation in TimelineTab.vue to enable contextual messaging for case events.

  • Added getEventDescription function with special handling for the "Case created" event to include visibility details.
  • Updated the template to use getEventDescription in place of a static lookup.
Comments suppressed due to low confidence (1)

src/dispatch/static/dispatch/src/case/TimelineTab.vue:73

  • [nitpick] Consider adding type annotations or JSDoc comments for the 'event' parameter to provide clearer API usage and improve developer experience.
const getEventDescription = (event) => {

@aaronherman aaronherman merged commit 20020d2 into main Jun 24, 2025
9 checks passed
@aaronherman aaronherman deleted the feat/include-visibility-in-timeline branch June 24, 2025 23:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants