Skip to content
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

Donald Trump can now decide the outcome of a battle #131

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

stefanospetrakis
Copy link

A button is presented to allowed roles/users, labeled "Trump It".
After pressing this button, D.T. will decide who wins and who loses and even explain it with a quote of his.

Stefanos Petrakis added 6 commits May 4, 2016 05:51
+ Wiki page => "Usage Importing a preexisting Drupal site"
+ Changed "How-to-import-a-preexisting-Drupal-site-into-kalastack" to "Usage-Importing-a-preexisting-Drupal-Site"
+ Setting the trumped field to be read-only
+ Setting the winner/loser and trumped value for the node
+ Setting the battle to 'completed' state
+ Adding conditions for when to run the trump-it action, at the moment only if the battle is not "completed"
@stefanospetrakis stefanospetrakis changed the title Donald Trump can now deciding on the outcome of a battle Donald Trump can now decide the outcome of a battle May 4, 2016
@pirog
Copy link
Contributor

pirog commented May 4, 2016

OMG!

@RobLoach
Copy link
Contributor

RobLoach commented May 6, 2016

Brilliant, well done, @stefanospetrakis. 👍

While adding the module is fantastic, and it's looking great, deploying it will require a manual enable of the module. Mind adding an update hook to one of the Playbox modules so that when it's deployed, it's automatically enabled? Thanks!

@pirog
Copy link
Contributor

pirog commented May 6, 2016

Can't wait for this feature!

MAKE PLAYBOX GREAT AGAIN

+ Related pull request: kalamuna#131
+ Related comment:  "... deploying it will require a manual enable of the module. Mind adding an update hook to one of the Playbox modules so that when it's deployed, it's automatically enabled?"
@stefanospetrakis
Copy link
Author

Hey @RobLoach, thanks for the thumbs up. Just commited my reaction to your comment ;)
Cheers!

@andrewmallis
Copy link
Member

@RobLoach – mulidev?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants