Flagsmith / Flagsmith/flagsmith

Standardise querying API calls count

Open
#6,985 1 comment 0 reactions 0 assignees View on GitHub
tech-debt
Dominant language
Python
Stars
6.6k
Forks
567
Avg merge
1d 13h
Merged PRs (30d)
121

Description

We've had `influxdb_wrapper.get_events_for_organisation`, which served the purpose of obtaining usage from organisations in SaaS. Then we introduced `influxdb_wrapper.get_current_api_usage` (https://github.com/Flagsmith/flagsmith/pull/3340), which... seemingly serves the same purpose. Both their code paths lead to almost the same communication with InfluxDB, except for some aggregation in Influx vs in Python.

https://github.com/Flagsmith/flagsmith/pull/6975 has introduced a flag `get_current_api_usage_deprecated` to help transition SaaS users to standard `get_events_for_organisation`.

## Acceptance criteria

- [ ] 100% of users are standardised without issues.
- [ ] The feature flag `get_current_api_usage_deprecated` is safely deleted.
- [ ] Code is updated to dismiss `get_current_api_usage`

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.