Skip to content

Remove the /export endpoint #2990

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 6 commits into from
Mar 7, 2023
Merged

Conversation

Dakostu
Copy link

@Dakostu Dakostu commented Mar 6, 2023

This PR removes all /export REST endpoints, with formatting functionality ported over to the /query endpoint.

@Dakostu Dakostu added refactoring Restructuring of existing code enhancement labels Mar 6, 2023
@Dakostu Dakostu requested a review from a team March 6, 2023 13:22
@Dakostu Dakostu force-pushed the topic/remove-the-export-endpoint branch from 3e0607f to cbecfa5 Compare March 6, 2023 13:22
@Dakostu Dakostu force-pushed the topic/remove-the-export-endpoint branch from cbecfa5 to e19103b Compare March 6, 2023 13:23
@Dakostu Dakostu marked this pull request as ready for review March 6, 2023 13:51
@Dakostu Dakostu force-pushed the topic/remove-the-export-endpoint branch from 7463612 to e6a5af9 Compare March 6, 2023 13:55
@Dakostu Dakostu force-pushed the topic/remove-the-export-endpoint branch from e6a5af9 to 946f6d9 Compare March 6, 2023 13:58
Copy link
Member

@lava lava left a comment

Choose a reason for hiding this comment

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

Looks good to me

@Dakostu Dakostu enabled auto-merge March 7, 2023 08:03
@Dakostu Dakostu disabled auto-merge March 7, 2023 08:26
@Dakostu Dakostu merged commit 3898c1c into master Mar 7, 2023
@Dakostu Dakostu deleted the topic/remove-the-export-endpoint branch March 7, 2023 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactoring Restructuring of existing code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants