-
Notifications
You must be signed in to change notification settings - Fork 65
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
fix phase #60
base: 7.x-4.x
Are you sure you want to change the base?
fix phase #60
Conversation
/.gitk-tmp.11456/ |
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.
@israelLinnovate @nevos Can you make this more generic? (for example .gitk-tmp*
)
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.
although it's not at all relevant to the PR..
@@ -158,7 +158,9 @@ function _ideal_blocks_idea_stats() { | |||
|
|||
// Hardcode phase for now. | |||
$phase = t('Phase'); | |||
|
|||
if (!empty($entity->field_in_action)){ | |||
$phase = $entity->field_in_action["und"][0]["value"]; |
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.
Make sure this string is safe (escaped as necessary)
@Israel123
@@ -158,7 +158,9 @@ function _ideal_blocks_idea_stats() { | |||
|
|||
// Hardcode phase for now. | |||
$phase = t('Phase'); |
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.
We need to have a default Phase, probably through the field's configuration.
For now, change the initial value to "Ideation"
@israelLinnovate
OI-39: #comment Added condition to check if voted the same user that created the idea.
OI-298: Fix notifications bug.
No description provided.