AwoRewards suggestions


  • Default avatar
    comenaranjas    
     8 years ago  last edited 8 years ago
    0

    Hi


    some ideas for this great component:


    - Highligh active coupons in payment history page.

    Is great that all historical data is there but coupons active and ready for use could be highlited somehow.. either black colour.. a column saying if is used or not.. for easy use of customers.

      

    - Please align column headers with values: visible in frontend "Payment history" and  backend.


    - Ability to hide frontpage tabs (menu items): Currently 4 are shown no matter which one you choose. Only way I found to hide 2 which we dont use was with language overrides.


    - Highlight "Unclaimed points".

     "total points" could lead to confusion since that is the first value that appears in "Credit History" page. I think "Unclaimed points" should be top there cause "total points" are the total number of points won which is something interesting but not as the current available points I guess.


    -Points module doesnt show for unlogged, a message like "please log in to see your points would be nice"

    Anyway just some ideas.


    Regards

  • Default avatar
    iRescue.gr    
     8 years ago
    0

    I totally agree with the above suggestions.
    We also tried to make some changes as we too wanted the user to be able to know which coupons are available and which aren't

    So in the components/com_aworewards/models/payment we change the query:

    $sql = 'SELECT p.id,p.user_id,p.coupon_id,p.amount_paid,p.payment_date,u.username,p.payment_type,p.payment_details,c.published

    FROM #__aworewards_payment p

    JOIN #__users u ON u.id=p.user_id

    JOIN #__awocoupon c ON c.id=p.coupon_id

    WHERE p.estore="'.AWO_ESTORE.'" AND u.id='.(int)$user->id.'

    '.$orderby;

    in thecomponents/com_aworewards/views/payment/tmpl/default.php we add::

    if ($row->published == -1) {

        $couponStatus = ' - <span class="couponInactive">'.JText::_('COM_AWOREWARDS_COUPON_STATUS_INACTIVE').'</span>';

    }

    And in the same place we change the coupon code:

    <td align="right">

    <?php
     
    echo
    $row->str_details.$couponStatus
    ?>
    </td>


    Apparently the changes in the default.phpare just suggestions and may not reflect the exact design that seyi wants

  • Default avatar
    comenaranjas    
     8 years ago
    0

    Thanks for reply and the code,

    Yeah from all the sugestions I think that is important that customers know which coupons are used and which ones aren't. We are launching the system with AwoRewards in few weeks and we already know that customers will be confused by the coupon list ;)

    Lets hope Seyi gets some time to consider it,

    Regards
  • Your avatar
    seyi    
     7 years ago
    0

    Hello,

    In the latest, 2.3.2, most of these are implemented.
  • Default avatar
    comenaranjas    
     7 years ago  last edited 7 years ago
    0

    Wow Seyi thank you very much. Looks fantastic and thanks for the multilingual interface, makes it so easy now to edit languages and the crossover text for used coupons is superb.

    Thanks for adding text to points module, just a little detail:

    modules/mod_aworewards_points/tmpl/login.php

    <div class="aworewards_points">
        <span>
    <?php
     
    echo $points->unclaimed
    ?>
    </span>
        <div>
    <?php
     
    echo JText::_('COM_AWOREWARDS_POINTS'
    ?>
    </div>
        Please log in
    </div>
    <div class="clear"></div>
    <br />

    "Please log in" is missing a language key so it can be translated or don't know if you added it that way for a reason.


    Regards
  • Default avatar
    iRescue.gr    
     7 years ago
    0

    Nice! I'm glad I could be of help!
    Thanks for the support seyi!
  • Your avatar
    seyi    
     7 years ago  last edited 7 years ago
    0

    Hello,

    Yes, the 'Please log in' is an oversitght.  For now, you can change
    Please log in

    to
    <?php echo JText::_('COM_AWOREWARS_PLEASELOGIN'); ?>


    Then in the backend, go to language override and setup the constant COM_AWOREWARS_PLEASELOGIN for all your languages.


    Actually never mind that, the 'Please log in' text already exists.  So change it to
    <?php echo JText::_('COM_AWOREWARDS_ERR_LOGIN'); ?>


    Then can make modifications as needed.
  • Default avatar
    comenaranjas    
     7 years ago
    0

    Solved, Thanks!