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

[9.0][auto_backup] Incorrect menu entry choice #726

Closed
pedrocs-exo opened this issue Feb 4, 2017 · 5 comments
Closed

[9.0][auto_backup] Incorrect menu entry choice #726

pedrocs-exo opened this issue Feb 4, 2017 · 5 comments
Labels
Milestone

Comments

@pedrocs-exo
Copy link
Contributor

auto_backup is UNUSABLE on v9:

auto_backup creates a menu entry under 'General Settings'. But by doing so, it disables the General Settings menu itself, preventing access to such important settings as mail gateways, multicompany setup, google integration and more. I can prepare a PR but need advice on the new menu location. Perhaps 'Technical/Database Structure'?

On a side note, the module description mentions "Do you want to know if the database backup succeeded or failed? Subscribe to the corresponding backup setting notification type." How can I do this subscription?

@lasley
Copy link
Contributor

lasley commented Feb 4, 2017

@pedrocasi - that menu location sounds fine.

Notifications are in Settings => Technical => Email => Followers => New

image

@lasley lasley added the bug label Feb 4, 2017
@lasley lasley added this to the 9.0 milestone Feb 4, 2017
@astirpe
Copy link
Member

astirpe commented Feb 8, 2017

I confirm the menu issue: it also occurs on V10.
Moving the menu to 'Technical/Database Structure' would be nice.

@pedrocs-exo
Copy link
Contributor Author

PR #728 fixes the issue on 9.0. I'll try to correct 10.0 as well as soon as possible. Thank you both!

And also a special thanks to Dave for the email followers tip!

@astirpe
Copy link
Member

astirpe commented Feb 14, 2017

Porting the fix to V10, here is the PR: #735

@lasley
Copy link
Contributor

lasley commented Feb 14, 2017

Closing to track in PRs

@lasley lasley closed this as completed Feb 14, 2017
SiesslPhillip pushed a commit to grueneerde/OCA-server-tools that referenced this issue Nov 20, 2024
Syncing from upstream OCA/server-tools (13.0)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants