Skip to content
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

Ensure composer.json exists #4121

Merged
merged 1 commit into from
Nov 11, 2016
Merged

Ensure composer.json exists #4121

merged 1 commit into from
Nov 11, 2016

Conversation

dangron
Copy link

@dangron dangron commented Apr 12, 2016

Ensure composer.json exists before json encoding suggests. The current implementation forces all modules to have a composer.json file or they cannot install sample data.

I discovered this issue when I tried to install sample data after creating my own hello world module.

@dangron
Copy link
Author

dangron commented Apr 12, 2016

I just saw issue #3400 where there is a much wider discussion going on, but this is still annoying :)

@sshrewz sshrewz added the PS label Apr 13, 2016
@mazhalai
Copy link
Contributor

@DanK00 can you please resync with develop and restart builds?

@mazhalai mazhalai self-assigned this Apr 20, 2016
Before json encoding suggests. The current implementation forces
all modules to have a composer.json file or they cannot install sample
data.
@dangron
Copy link
Author

dangron commented Apr 21, 2016

@mazhalai done :)

@sshrewz
Copy link

sshrewz commented Aug 11, 2016

Internal ticket - MAGETWO-56786

@sshrewz sshrewz added the linked label Aug 11, 2016
@vkorotun vkorotun added Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development and removed linked labels Aug 22, 2016
@mmansoor-magento mmansoor-magento merged commit 6dd36ba into magento:develop Nov 11, 2016
mmansoor-magento pushed a commit that referenced this pull request Nov 11, 2016
Fixed issues:
- MAGETWO-60647: Delivery of bug fixes for Sample Data and Import/Export
- MAGETWO-56787: [GITHUB][PR] Added call to action to compile command error #4134
- MAGETWO-56786: [GITHUB][PR] Ensure composer.json exists #4121
- MAGETWO-57799: cannot upgrade 2.0 => 2.1 with auto_increment > 1
- MAGETWO-59715: Impossible to import additional_images with labels which have a comma separator
- MAGETWO-60633: [Github] .htaccess deny code execution not working for Apache + php-fpm #6766
- MAGETWO-46636: Nginx doesn't redirect to setup page when using port
@BaDos
Copy link
Contributor

BaDos commented Nov 11, 2016

Merged.
Thank you for your contribution.

magento-engcom-team pushed a commit that referenced this pull request Apr 26, 2019
[TSG] MTF fixes for 2.3 (pr15) (2.3.2)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Component: Setup Issue: Ready for Work Gate 4. Acknowledged. Issue is added to backlog and ready for development Progress: accept
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants