On the admin page for this module, the merchant can set a value of max days to return an order,
why are you using a long text field, if the merchant can just set a short value (between 1 and ?999?)
url: admin/commerce/config/return

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

GuGuss’s picture

Title: Admin page : RAM Settings field format » Revamp the Return Settings configuration page
FileSize
7.02 KB

Hi,

I think it could go even further !

Here are some improvements that could greatly improve the usability of the module:

  • Remove the "Activate returns" checkbox since we the user can either leave empty or enter 0 if returns are not needed.
  • The "Return period" shouldn't be a single textfield. It'll be better to display a value and a select list with settings like: Days, Weeks, Months, Years. See the attached screenshot as an example.
  • The return period shouldn't be global to the site. It could be on the product type configuration page (see the issue: #2027459: Define if a product can be return or not) or specific to an order type.
jkuma’s picture

Assigned: Unassigned » jkuma
Status: Active » Needs work

why are you using a long text field,

Ok !

Remove the "Activate returns" checkbox since we the user can either leave empty or enter 0 if returns are not needed.

This functionality can't be removed because the administrator can disable temporally RMA and still manage the returns.

The "Return period" shouldn't be a single textfield. It'll be better to display a value and a select list with settings like: Days, Weeks, Months, Years. See the attached screenshot as an example.

The return period shouldn't be global to the site. It could be on the product type configuration page (see the issue: #2027459: Define if a product can be return or not) or specific to an order type.

Ok !

jkuma’s picture

Status: Needs work » Needs review
FileSize
8.42 KB

Here is an attempt to solve this feature request. I've also added an "order status" setting which determine when the return can be created. (snapshot)

jkuma’s picture

For some reasons, the patch was not correctly uploaded...

jkuma’s picture

jkuma’s picture

Status: Needs review » Fixed

Committed in 7.x-1.x-dev branch.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

  • Commit 7e220df on 7.x-1.x, 7.x-2.x, 7.x-2.x-goldorak by goldorak:
    Issue #2027443 by goldorak | asplamagnifique: Revamp the Return Settings...