Skip to content

Conversation

@zamandevs
Copy link
Contributor

No description provided.

@FaraziF FaraziF added the needs: dev review This PR needs review by a developer label Nov 28, 2020
$custom = json_decode( stripcslashes( $postdata['custom'] ) );
$coupon_id = isset( $custom->coupon_id ) ? $custom->coupon_id : false;

if ($postdata['period3']) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@zamandevs Use space like if ( $postdata['period3'] )

$coupon_id = isset( $custom->coupon_id ) ? $custom->coupon_id : false;

if ($postdata['period3']) {
update_user_meta($custom->user_id, '_wpuf_used_trial', 'yes');
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use spae update_user_meta( $custom->user_id, '_wpuf_used_trial', 'yes' )

@FaraziF FaraziF added the bug label Dec 3, 2020
@FaraziF FaraziF added QA Approved This PR is approved by the QA team and removed bug needs: testing labels Dec 7, 2020
@zamandevs zamandevs added Dev Review Done and removed needs: dev review This PR needs review by a developer labels Dec 7, 2020
[
'total_items' => $total_items,
'per_page' => $per_page,
]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Indentation problem.

@tareq1988
Copy link
Member

Please use editorconfig to fix the spacing problem on both the files.

@tareq1988 tareq1988 added the needs: author reply This PR needs author feedback or code changes label Dec 7, 2020
@tareq1988 tareq1988 merged commit 8dd8a6b into weDevsOfficial:develop Dec 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Dev Review Done needs: author reply This PR needs author feedback or code changes QA Approved This PR is approved by the QA team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants