One of the things I liked most in this module when compared with similar modules for environment tools was the depth of documentation provided, especially the video on configuration. The entire thing works effectively and is easy to setup, the only thing I'd like to mention is that the routing.yml file should be updated as pattern has been changed in Symfony for path.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mimes’s picture

mimes’s picture

[please ignore/remove this comment, submitted twice accidentally]

mimes’s picture

Status: Active » Needs review
amitgoyal’s picture

The suggested changes in the patch are now there in the latest 8.x-2.x branch.

e0ipso’s picture

Actually any issue with the Drupal 8 branch will be dealt with when D8 is in a more mature stage.

amitgoyal’s picture

@e0ipso - That makes sense but when people see 8.x branch for modules then they try to use them and like to provide fixes if required. :)

So may be we should give them their due effort by accepting their patches and closing the issues!

mparker17’s picture

Status: Needs review » Fixed

As of the latest commit to the 8.x-3.x branch (3132a40 on 2015-11-07), this is done, so marking as fixed.

Edit: I didn't notice this issue was filed against the 8.x-2.x branch. That being said, it seems like Drupal 8 development is happening on the 8.x-3.x branch.

Status: Fixed » Closed (fixed)

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