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

Issue Fixed Related to Delete List Items from auto export Report setting and associated config setting for Cron. #40

Merged
merged 7 commits into from
Mar 3, 2023

Conversation

Ravikr139
Copy link
Contributor

Issue Fixed to remove auto export Report list and config setting for Cron.

@Ravikr139
Copy link
Contributor Author

Issue Fixed to remove auto export Report list and config setting for Cron.
@peterjaap

@peterjaap
Copy link
Contributor

@Ravikr139 what issue are we talking about?

@Ravikr139
Copy link
Contributor Author

@peterjaap issue specially related to the deletion of existing automated export settings (Report->Automated Exports).
It was not working there.
This issue has been fixed in this.

@Ravikr139 Ravikr139 changed the title Issue Fixed to remove auto export Report list and config setting for Cron. Issue Fixed Related to Delete List Items from auto export Report setting and associated config setting for Cron. Sep 26, 2022
@peterjaap
Copy link
Contributor

Right, thanks!

@peterjaap
Copy link
Contributor

I'm not the maintainer however, so I can't merge this. LGTM however.

@jantzenw @brobie

@Ravikr139
Copy link
Contributor Author

@jantzenw @brobie

Please Review and merge this PR.
It will fix the major issue related to delete export list setting.

@jantzenw jantzenw merged commit b51c2ac into degdigital:master Mar 3, 2023
@jantzenw
Copy link
Collaborator

jantzenw commented Mar 3, 2023

This PR is being merged. This issue is also fixed in v3.1.1+ (albeit slightly differently).

Side note, the master branch is out of date. It is purposely not being updated much because this repo lacked tags for many years, causing some users to install the extension as "dev-master", so updates to the master branch could have serious consequences.

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.

3 participants