#25 Changes "Refresh" to "Reharvest" and adds confirmation dialog #26
Conversation
ghost
assigned 
|
Hey @amercader can you take a look? Want this in the package today |
| {% else %} | ||
| <li>{{ h.nav_named_link(_('Refresh'), '{0}_refresh'.format(c.dataset_type), id=source.id, class_='btn', icon='refresh')}}</li> | ||
| {% set locale = h.dump_json({'content': _('This will re-run the harvesting for this source. Any updates at the source will overwrite the local datasets. Large collections may take a couple of days to finish harvesting. Please confirm you would like us to start reharvesting.')}) %} |
There was a problem hiding this comment.
Large collections may take a couple of days to finish harvesting.
This is not relevant to a default ckanext-harvest install and will create a lot of confusion, can we drop it here and add a custom admin_base.html file in ckanext-geodatagov with the message you prefer there?
There was a problem hiding this comment.
@amercader Sure. So we have the message like: (for ckanext-harvest)
This will re-run the harvesting for this source. Any updates at the source will overwrite the local datasets. Please confirm you would like to start reharvesting.
And then on ckanext-geodatagov:
This will re-run the harvesting for this source. Any updates at the source will overwrite the local datasets. Large collections may take a couple of days to finish harvesting. Please confirm you would like to start reharvesting.
|
@amercader OK, there you go. I'll update https://github.com/okfn/ckanext-geodatagov with an issue/pull request in a bit. |
Fixes #25