Skip to content
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

Add: New multi_selection report format param type #2165

Merged
merged 3 commits into from
May 17, 2024

Conversation

timopollmeier
Copy link
Member

What

A new parameter type for report formats and report configs is added that allows selecting multiple predefined options.

Why

This can later be used to add new customization options to report formats like a column selection for CSV.

References

GEA-403

@timopollmeier timopollmeier requested a review from a team as a code owner April 10, 2024 09:33
Copy link

Conventional Commits Report

Type Number
Added 1

🚀 Conventional commits found.

@timopollmeier timopollmeier force-pushed the report-format-multi-select-params branch 2 times, most recently from 69a3f46 to a166ad7 Compare April 22, 2024 06:42
@timopollmeier timopollmeier marked this pull request as draft April 25, 2024 12:35
A new parameter type for report formats and report configs is added that
allows selecting multiple predefined options.

This can later be used to add new customization options to report
formats like a column selection for CSV.
@timopollmeier timopollmeier force-pushed the report-format-multi-select-params branch from a166ad7 to 65cd021 Compare April 25, 2024 14:18
src/manage_sql_report_formats.c Show resolved Hide resolved
src/manage_sql_report_formats.c Show resolved Hide resolved
src/manage_sql_report_formats.c Show resolved Hide resolved
@timopollmeier timopollmeier marked this pull request as ready for review May 17, 2024 08:36
@timopollmeier timopollmeier merged commit 4f29ca4 into main May 17, 2024
9 checks passed
@timopollmeier timopollmeier deleted the report-format-multi-select-params branch May 17, 2024 08:36
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