Skip to content

Conversation

@litury
Copy link
Contributor

@litury litury commented Jan 29, 2026

Fixed events_create tool - was using non-existent /i/data-manager/event endpoint.

Changed to /i/events/edit_map with correct event_map parameter format.

@ar2rsawseen ar2rsawseen force-pushed the fix/events-create-endpoint branch from 112c999 to c8d15e0 Compare January 29, 2026 15:51
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@litury
Copy link
Contributor Author

litury commented Feb 2, 2026

Updated the PR with test fixes:

  • Fixed 2 failing tests in tests/events.test.ts
  • Tests were expecting old endpoint /i/data-manager/event but implementation uses /i/events/edit_map
  • All tests now pass locally: ✅ 255 tests passed

The test failures were caused by the endpoint change in commit 112c999 where tests weren't updated. Now both implementation and tests are in sync.

Ready for review! 🚀

@ar2rsawseen
Copy link
Member

Hello,
sorry it took some time to check this.

there is some difference for this between Lite and Enterprise edition. It was created with enterprise in mind, but this endpoint should actually work for both, so we will merge it :)

@ar2rsawseen ar2rsawseen merged commit 214d1ba into Countly:main Feb 2, 2026
4 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.

2 participants