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

Show installed modules by default instead of the modules selection #8244

Merged
merged 1 commit into from Nov 14, 2017

Conversation

Projects
None yet
4 participants
@christianverardi
Contributor

christianverardi commented Aug 11, 2017

Questions Answers
Branch? develop
Description? Show installed modules first
Type? improvement
Category? BO
BC breaks? no
Deprecations? no
Fixed ticket? BOOM-3390
How to test? From the BO click on the Modules' page link from the menu.

This change is Reviewable

Cristiano Verardi
@aleeks

This comment has been minimized.

Show comment
Hide comment
@aleeks

aleeks Aug 11, 2017

Contributor

ping @vincentbz

Contributor

aleeks commented Aug 11, 2017

ping @vincentbz

@Quetzacoalt91

This comment has been minimized.

Show comment
Hide comment
@Quetzacoalt91

Quetzacoalt91 Aug 11, 2017

Member

The module section will be reworked. The entry point is going to change as written in this PR.

However, I won't merge this PR until the project has started.

Member

Quetzacoalt91 commented Aug 11, 2017

The module section will be reworked. The entry point is going to change as written in this PR.

However, I won't merge this PR until the project has started.

@Quetzacoalt91 Quetzacoalt91 added this to the 1.7.3.0 milestone Aug 11, 2017

@christianverardi

This comment has been minimized.

Show comment
Hide comment
@christianverardi

christianverardi Aug 11, 2017

Contributor

@Quetzacoalt91 ok, thanks for the feedback.

Contributor

christianverardi commented Aug 11, 2017

@Quetzacoalt91 ok, thanks for the feedback.

@LittleBigDev

This comment has been minimized.

Show comment
Hide comment
@LittleBigDev
Contributor

LittleBigDev commented Oct 9, 2017

@LittleBigDev LittleBigDev requested a review from Quetzacoalt91 Oct 9, 2017

@Quetzacoalt91

This comment has been minimized.

Show comment
Hide comment
@Quetzacoalt91

Quetzacoalt91 Oct 9, 2017

Member

As said, no merge until the project has started. 😉

Member

Quetzacoalt91 commented Oct 9, 2017

As said, no merge until the project has started. 😉

@Quetzacoalt91 Quetzacoalt91 modified the milestones: 1.7.3.0, 1.7.4.0 Oct 9, 2017

@Quetzacoalt91 Quetzacoalt91 changed the title from BO: Show installed module first to Show installed module first Nov 14, 2017

@Quetzacoalt91 Quetzacoalt91 merged commit 7500ce0 into PrestaShop:develop Nov 14, 2017

2 checks passed

codacy/pr Good work! A positive pull request.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Quetzacoalt91

This comment has been minimized.

Show comment
Hide comment
@Quetzacoalt91
Member

Quetzacoalt91 commented Nov 14, 2017

Thank you @christianverardi

@eternoendless eternoendless changed the title from Show installed module first to Show installed modules by default instead of the modules selection Apr 12, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment