-
Notifications
You must be signed in to change notification settings - Fork 5
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
Sync remote resources in groups edit #104
Conversation
This is the base for a cron service to be ran periodically. The local and remote resource tables are compared when the view is requested.
instead of a 'resource_id' To distinguish permission requests and 'clean_resource' requests
…nc-remote-resources-in-groups-edit
…' into sync-remote-resources-in-groups-edit
write svc_sync_type to db
Everything is working locally, but magpie on colibri is down because we need to update the providers.cfg file... see https://github.com/Ouranosinc/PAVICS/pull/103 |
magpie/register.py
Outdated
@@ -359,7 +359,8 @@ def magpie_register_services_with_db_session(services_dict, db_session, push_to_ | |||
for svc_name, svc_values in services_dict.items(): | |||
svc_new_url = os.path.expandvars(svc_values['url']) | |||
svc_type = svc_values['type'] | |||
svc_sync_type = svc_values['sync_type'] | |||
|
|||
svc_sync_type = svc_values.get('sync_type', svc_values['title']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe instead of 'title', let's default to None or other value reflecting not-implemented?
Routes like |
9b28317
to
f93d97a
Compare
Description
Clean
button to remove the resource from the database. There is also aClean all
button available.Supported services
Other notes
There is a direct database call from the UI in this PR. A note has been added in the code.