-
Notifications
You must be signed in to change notification settings - Fork 417
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
Ability to create predefined calendar or(and) addressbook for new user #1473
base: master
Are you sure you want to change the base?
Conversation
Good idea, but please check my comments. |
disabled by default is my example in documentation and config unclear? |
I had no time to test so far, but scheduled |
Can you please extend the code that it validates the config option during start? It would be helpful to avoid any broken config sniplet detected only on the "new user" trigger. |
Which aspect should I check? The json syntax is checked during json.loads in config.py. |
Best that they are valid collection properties (calendar or address book). Alternative: hardcode and only add an option for enabling of that feature, then the QA can be done by test script. |
check_and_sanitize_props called before create_collection in |
but this will be triggered on first usage only, and then it breaks....I would expect that "preloaded" config is assured working and not waiting to fail on first usage. |
No description provided.