Skip to content

Added debugging info for top events job#6119

Merged
Cookiezaurs merged 1 commit intorelease.24.10from
fixes.24-10
Apr 5, 2025
Merged

Added debugging info for top events job#6119
Cookiezaurs merged 1 commit intorelease.24.10from
fixes.24-10

Conversation

@Cookiezaurs
Copy link
Copy Markdown
Contributor

No description provided.

@kanwarujjaval kanwarujjaval requested a review from Copilot April 4, 2025 18:05
Copy link
Copy Markdown
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.

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (2)

api/jobs/topEvents.js:144

  • [nitpick] The loop variable 'z' is ambiguous; consider renaming it to a more descriptive name such as 'i' or 'appIndex'.
for (var z = 0; z < getAllApps.length; z++) {

api/jobs/topEvents.js:209

  • The method name 'eventsCollentions' appears to be misspelled; it is likely intended to be 'eventsCollections'.
const collectionNameEvents = this.eventsCollentions({ event, id: app._id });

@Cookiezaurs Cookiezaurs merged commit 80976c9 into release.24.10 Apr 5, 2025
5 of 8 checks passed
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