Closed (fixed)
Project:
Hosting
Version:
6.x-2.x-dev
Component:
User interface
Priority:
Minor
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
29 Aug 2012 at 10:46 UTC
Updated:
11 Oct 2014 at 04:11 UTC
Jump to comment: Most recent
Comments
Comment #1
helmo commentedThe verify task is triggered automatically right?
Your suggestion might leave a user wondering whether they should themselves trigger a verify task.
What about: "Site XYZ.com has been updated. A Verify task has been created to process any changes."
Comment #2
ergonlogicComment #3
ken hawkins commentedHad forgotten about this issue. The suggestion by Helmo in #1 sounds like an improvement to me.
Comment #4
ergonlogicThis isn't just a string change, as this message is generated by the node module in core. Also, it applies to platform and server node save too. I'm inclined to add this message to all 3: "Any changes will take effect once the scheduled Verify task has been processed."
Comment #5
ergonlogicFixed in 4d1f29b
Comment #6
ergonlogicI reverted the above fix, since it was causing the message to be displayed much more often than it should have. New fix in 6d06b8f.
Comment #10
jon pughThis language doesn't make sense if installing a site or creating a new platform.
I've committed e97ce4 to add messaging like so:
Creation:
Editing:
After I did this, I couldn't help myself. I improved the Task schedule message as well. This drupal_set_message() call shouldn't be in hosting_add_task(), and it shouldn't be called if the hosting task queue is disabled.
See fef1ca
So now, only if you are actually using the queue, installing a site looks like this:
Comment #11
jon pughI still think we should get rid of the drupal core "@type %title has been created." message. It's not helpful here.