Skip to content

Fix JSDoc parameters for GET_AFFECTED_USERS_CHART_DATA action#822

Merged
slaveeks merged 2 commits intofeat/affected-users-chartfrom
copilot/sub-pr-821
Jan 21, 2026
Merged

Fix JSDoc parameters for GET_AFFECTED_USERS_CHART_DATA action#822
slaveeks merged 2 commits intofeat/affected-users-chartfrom
copilot/sub-pr-821

Conversation

Copy link
Copy Markdown

Copilot AI commented Jan 21, 2026

The JSDoc for GET_AFFECTED_USERS_CHART_DATA documented parameters (startDate, endDate, groupBy) that don't exist in the actual function signature.

Changes:

  • Updated JSDoc to reflect actual parameters: projectId, eventId, originalEventId, days
  • Removed incorrect parameters and unused context.dispatch reference
  • Changed parameter object name from "project" to "payload" for clarity

Before:

/**
 * @param project.startDate - start date
 * @param project.endDate - end date
 * @param project.groupBy - grouping interval in minutes
 */
async [GET_AFFECTED_USERS_CHART_DATA](
  { commit },
  { projectId, eventId, originalEventId, days }

After:

/**
 * @param payload.projectId - project's id
 * @param payload.eventId - event's id
 * @param payload.originalEventId - original event's id
 * @param payload.days - number of days to fetch chart data for
 */
async [GET_AFFECTED_USERS_CHART_DATA](
  { commit },
  { projectId, eventId, originalEventId, days }

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

@codex-assistant codex-assistant bot marked this pull request as ready for review January 21, 2026 17:35
@codex-assistant
Copy link
Copy Markdown

Thanks for adding a description — the PR is now marked as Ready for Review.

Co-authored-by: n0str <988885+n0str@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix chart data implementation in affected-users feature Fix JSDoc parameters for GET_AFFECTED_USERS_CHART_DATA action Jan 21, 2026
Copilot AI requested a review from n0str January 21, 2026 17:39
@slaveeks slaveeks merged commit 7c16d33 into feat/affected-users-chart Jan 21, 2026
@slaveeks slaveeks deleted the copilot/sub-pr-821 branch January 21, 2026 17: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.

3 participants