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 flag to 'Upsert-Reports' command for optional overwrite #6298

Closed
fergie-nz opened this issue Jan 28, 2025 · 1 comment
Closed

Add flag to 'Upsert-Reports' command for optional overwrite #6298

fergie-nz opened this issue Jan 28, 2025 · 1 comment
Labels
Build Tested: None This issue cannot be build tested or will not be build tested enhancement New feature or request feature: reports Team Ruru 🦉 Roxy, Ferg, Noel
Milestone

Comments

@fergie-nz
Copy link
Contributor

Is your feature request related to a problem? Please describe 👀

Upsert command uses an insert functionality by checking for existing report ids before upserting.
Would make for faster development if we could pass an optional overwrite (-o?) flag so that developers can quickly test iterations of new reports.

This would also be useful for testing - the upserting of reports on startup of our backend uses the same cli function as 'upsert-reports'.

Adding an optional overwrite flag would cover both use cases enabling fast development of a specific report, and fast overwrite for testers when testing standard reports.

Describe the solution you'd like 🎁

Add an optional '-o' to the 'upsert-report' command.

If this is added, don't check for if the report exists on upsert.

Describe alternatives you've considered 💭

Additional context 💌

Moneyworks Jobcode 🧰

OMS:REPT

@fergie-nz fergie-nz added enhancement New feature or request needs triage labels Jan 28, 2025
@fergie-nz fergie-nz added this to the v2.6.0 milestone Jan 28, 2025
@andreievg
Copy link
Collaborator

duplicate of #5863

@marthakeezy marthakeezy added the Build Tested: None This issue cannot be build tested or will not be build tested label Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Tested: None This issue cannot be build tested or will not be build tested enhancement New feature or request feature: reports Team Ruru 🦉 Roxy, Ferg, Noel
Projects
None yet
Development

No branches or pull requests

3 participants