@abb says, And that’s where you come in, mate! You and Jo should push this issue forward and I’ll support you guys.
Like, sweetums, grab your pink tuutuu, white stockings, and blue pom poms and move along to the sidelines of life and cheer! Give me a J! Give me an O! Cheer! Cheer!
You ready to put on your big boy panties and do some real work? No? Then leave.
Yes? You want to contribute but you can’t program? Then waddle over here my lazy entitled big boy panties wearing around your ankles and get something under those fingernails besides boogers, like keystrokes that have purpose and meaning:
@abb says, How can we actually get this implemented?
You asked, I’m delivering, and we here expect you to step up. Smash bugs: https://github.com/mautic/mautic
How to test a pull request
Everyone can test submitted features and bug fixes. No programming skills are required (ahem, @abb) . All you have to do is to follow the steps below.
Install the latest GitHub version
- Open a Terminal/Console window.
- Change directory to the server root (i.e. cd /var/www if your local server root is at /var/www).
- Clone the repository (git clone https://github.com/mautic/mautic.git)
- The mautic directory should appear in the server root. Change directory to mautic directory (cd mautic).
- Install dependencies (composer install).
- Visit Mautic in a browser (probably at http://localhost/mautic) and follow installation steps.
Development Environment
Mautic downloaded from GitHub have the development environment. You can access it by adding index_dev.php after the Mautic URL. Eg. http://localhost/mautic/index_dev.php/s/. Or in case of CLI commands, add --env=dev attribute to it.
This development environment will display the PHP errors, warnings and notices directly as the output so you don’t have to open the log to see them. It will also load for example translations without cache, so every change you make will be visible without clearing it. The only changes which requires clearing the cache are in the config.php files.
In case of assets like JS, CSS, the source files are loaded instead of concatenated, minified file. This way the changes in those files will be directly visible on refresh. If you’d want to see the change in production environment, you’d have to run the app/console mautic:assets:generate command.
In many cases, the CSS files are built from LESS files. To compile the changes in the LESS files, run grunt compile-less command.
Test a Pull Request (PR)
Every change to Mautic core happens via PRs. Every PR must have 2 successful tests to be merged to the core and released in the next version. Testing a PR is a great way how to move Mautic forward and personally improve its quality and stability.
Select a PR to test.
Read the description and steps to test. If it’s a bug fix, follow the steps if you’ll be able to recreate the issue.
Use the development environment (above) for testing.
Apply the PR
Clear cache for development environment (rm -rf app/cache/* or app/console cache:clear -e dev).
Follow the steps from the PR description again to see if the result is as described.
Write a comment how the test went.
If there is a problem, provide as many information as possible including error log messages.
Or, @abb, you can cheer in your pink tuu tuu on the sidelines of life while we get real work done.