Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Our client is confused by the whole draft/published and deploy/checked/unchecked interface.
I know that ptsimard has created a wonderful ajax solution to make the interface more user friendly with deploy checkbox and draft/published options . Our client wants this functionality, would it be possible to share this with us ? Otherwise I might have to re-engineer the same type of thing from scratch?
Comment | File | Size | Author |
---|---|---|---|
#11 | wetkit_deployment_source-improve-ui-experience-2631186-11.patch | 11.6 KB | joseph.olstad |
Comments
Comment #2
joseph.olstadBasically user interface enhancement, when the published is changed to draft, the ajax drupalBehavior is triggered and warns the end user what the consequences of this action are by changing the colors and adding help text drawing their attention to the instructions making it very clear to what happens.
Comment #3
joseph.olstadwithout this interface, initial feedback from our client is that they don't want any deployments to happen outside of batch deployment. I'm not sure that they understand the benefit of being able to unpublish something from the production site (in case it was an error in publishing, etc) . I think this would be easier to understand with the ptsimard solution.
One other thing, is there a way to disable the deployment of users when they change passwords? we may want to have different admin passwords on source and destination but this isn't possible if autodeploy is happening for users.
basically the ptsimart deployment interface solution might help us sell the default deployment configuration, maybe if it was easier to understand they'd buy in.
Comment #4
joseph.olstadIf someone wants to send me their custom code for this 'hint hint', I'll refactor it for the distro, or else just modify it for our use.
Comment #5
joseph.olstadOur clients really prefer using the queue.
We've configured the variable shared_content_source_deployment_disabled TRUE
drush vset shared_content_source_deployment_disabled TRUE
this allows us to deploy using the Queue and nothing but the queue. I'm wondering if we need to adjust our shared_content boolean settings , but so far deployment seems ok, with exception of field collections.
Comment #6
sylus CreditAttribution: sylus commentedPostponing this till @ptsimard is back from vacation as is his functionality and will need some small changes to be ported upstream.
Mostly related to content types that don't have workbench moderation but are deployable.
Comment #7
joseph.olstadIn addition to the ptsimard solution, I propose some additional user interface enhancements:
/admin/content
CATEGORY TITLE TYPE AUTHOR PBLSHD
News Test deploy Basic admin YES
CONTUED header
LANG UPDATED DEPLOY DEPLOYED
Eng Dec16 14:00 yes no
UPDATED date time format: CUSTOM
Md G:i
NOW for the dashboard/deployment/deploy node view:
same as above, except add filter for deploy to destination, only show if YES
use trim to trim language to 3 characters
use trim to trim categories to 10 characters
rename headers so that this fits nicely.
I've done this, makes content editing much better.
On the deployment bean view, similar thing, should only show beans that are marked for deployment. (filter on those)
a column for DEPLOYED needs to be added there on bean view for deployment as well. admin/content/ bean one too should have that
this will prevent mistakes and also will improve usability.
Comment #8
joseph.olstadSee my previous comment, this can be done while we wait for ptsimard, I Have done this in my local setup, seems like good default behavior and improves usability imho
Comment #9
sylus CreditAttribution: sylus commentedCan you submit the patch and i'll take a look ^_^ makes sense to improve the interface.
Comment #10
joseph.olstadwhen I have a moment I'll submit the patch.
Comment #11
joseph.olstadHere's the patch .
TODO: The binary format for Deploy and Deployed should be formated as a Yes/No instead of 1/0
so we'll have to add a views format handler for that as we did for changed field from unix time to human time.
and speaking of which: There is one regression with this patch, to do with the changed date format handler. We should fix that first before commit
see http://cgit.drupalcode.org/wetkit_deployment/commit/?id=1f4ff7c8c819dadd...
Comment #12
joseph.olstadFollowup:
1) tried to format boolean as yes-no but having issues, here's the code I added to wetkit_deployment_source_views_data_alter (but it doesn't work)
2) just a note: wetkit_deployment_source_views_data_alter does not need to return $data
because $data is passed in by reference.
Comment #13
joseph.olstadI think the problem is related to the view formatter being 'Key' , maybe there is a special way to handle formatting 'Key' for boolean values , not sure why it'd be different.
Comment #14
sylus CreditAttribution: sylus commentedI am going to close this as have the new deploy plus interface and it seems more people are leveraging the bulk deploy interface then shared content.
Comment #15
joseph.olstadsounds good.