Skip to content

typo #10327

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Feb 19, 2025
Merged

typo #10327

merged 1 commit into from
Feb 19, 2025

Conversation

guillim
Copy link
Contributor

@guillim guillim commented Feb 19, 2025

typo when doing the fix

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

PR Summary

Fixed a typo in the mock data for views, changing the aggregate operation from uppercase 'Count' to lowercase 'count' to maintain consistency.

  • Updated AGGREGATE_OPERATIONS.Count to AGGREGATE_OPERATIONS.count in /packages/twenty-front/src/testing/mock-data/views.ts to match the enum value used throughout the codebase

1 file(s) reviewed, no comment(s)
Edit PR Review Bot Settings | Greptile

@Devessier
Copy link
Contributor

Thanks for the fix, I was going to work on it.

@guillim guillim enabled auto-merge (squash) February 19, 2025 13:48
@guillim guillim merged commit 7636def into main Feb 19, 2025
49 of 50 checks passed
@guillim guillim deleted the ci branch February 19, 2025 13:52
Copy link
Contributor

Thanks @guillim for your contribution!
This marks your 71st PR on the repo. You're top 1% of all our contributors 🎉
See contributor page - Share on LinkedIn - Share on Twitter

Contributions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants