Skip to content

Conversation

@lukewhchen
Copy link
Contributor

@lukewhchen lukewhchen commented Oct 15, 2025

Summary

Add more timeframe options: 90d, 180d, 365d, current_year and previous_year

Ticket: https://konghq.atlassian.net/browse/KHCP-18261

@lukewhchen lukewhchen self-assigned this Oct 15, 2025
@lukewhchen lukewhchen changed the title fix(analytics): add more timeframe keys feat(analytics): add more timeframe option keys Oct 15, 2025
@lukewhchen lukewhchen changed the title feat(analytics): add more timeframe option keys feat(analytics): add more timeframe options Oct 15, 2025
@lukewhchen lukewhchen marked this pull request as ready for review October 15, 2025 20:46
@lukewhchen lukewhchen requested a review from a team as a code owner October 15, 2025 20:46
}
}

class PerviousYear extends Timeframe {
Copy link
Contributor

Choose a reason for hiding this comment

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

Minor: typo: 'Previous'

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

filipgutica
filipgutica previously approved these changes Oct 16, 2025
@filipgutica filipgutica dismissed their stale review October 16, 2025 18:06

missed typo

'30d',
'current_month',
'previous_month',
'90d',
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you need to add these here? Adding these would mess up some of our validations.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is for ts:
Screenshot 2025-10-16 at 11 14 40 AM

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Would you prefer that I use assertions?

@lukewhchen lukewhchen merged commit 5d00e85 into main Oct 16, 2025
13 checks passed
@lukewhchen lukewhchen deleted the feat-add-more-time-frame-key-options branch October 16, 2025 20:03
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