|
| 1 | +# Guidelines for Getting Help with UserFrosting |
| 2 | + |
| 3 | +**Before** you open a new issue or ask a question in chat, you **must** read these guidelines. If it is evident from your issue that you failed to research your question properly, your issue may be closed without being answered. |
| 4 | + |
| 5 | +## Troubleshooting |
| 6 | + |
| 7 | +There are a few common stumbling blocks that new users face when setting up UserFrosting for the first time. If you are new to the current version of UserFrosting, please first look at the [basic requirements and installation instructions](https://learn.userfrosting.com/basics/requirements/basic-stack). |
| 8 | + |
| 9 | +If you don't find what you're looking for in the troubleshooting page, then please check the [wiki](https://github.com/userfrosting/UserFrosting/wiki) and [existing issues](https://github.com/alexweissman/UserFrosting/issues?utf8=%E2%9C%93&q=is%3Aissue), both opened and closed. Your question may have already been asked and answered before! |
| 10 | + |
| 11 | +You can also search for help on Stack Overflow or in our [Forums](https://forums.userfrosting.com/). In addition to the tags for the components that UF builds upon, such as [Slim](http://stackoverflow.com/questions/tagged/slim), [Twig](http://stackoverflow.com/questions/tagged/twig), [Eloquent](http://stackoverflow.com/questions/tagged/eloquent), [jQuery Validate](http://stackoverflow.com/questions/tagged/jquery-validate), [Select2](http://stackoverflow.com/questions/tagged/jquery-select2), there is now a [UserFrosting tag](http://stackoverflow.com/questions/tagged/userfrosting) as well. |
| 12 | + |
| 13 | +There are also tags for the utilities upon which UserFrosting depends, such as [Composer](http://stackoverflow.com/questions/tagged/composer-php) and [Git](http://stackoverflow.com/questions/tagged/git). |
| 14 | + |
| 15 | +## Asking for Help |
| 16 | + |
| 17 | +In general, the Github issue tracker should only be used for bug reports and feature requests. If you're just having trouble getting something to work, you should ask on Stack Overflow or in our [Forums](https://forums.userfrosting.com/) instead. Tag your question with the `userfrosting` tag, and optionally with any tags specific to the relevant underlying technologies, such as `slim`, `twig`, `eloquent`, `composer`, etc. You should also mention the version of UserFrosting that you are using. |
| 18 | + |
| 19 | +After posting a question on Stack Overflow or in our [Forums](https://forums.userfrosting.com/), please [link to it in chat](https://chat.userfrosting.com). This will ensure that more people see it, and provide a place where we can discuss and help clarify your question. |
| 20 | + |
| 21 | +On Github, Chat, and Stack Overflow, please keep in mind the following: |
| 22 | + |
| 23 | +1. Remember that courtesy and proper grammar go a long way. Please take the time to craft a **precise, polite issue**. We will do our best to help, but remember that this is an open-source project - none of us are getting paid a salary to develop this project, or act as your personal support hotline :wink: |
| 24 | + |
| 25 | +2. Report any errors in detail. Vague issues like "it doesn't work when I do this" are not helpful. Show that you have put some effort into identifying the cause of the error. |
| 26 | + |
| 27 | +3. There are three main places where you may find error messages: |
| 28 | + |
| 29 | +- Backend (PHP-related) fatal errors: in your PHP error log. This is usually a file called `php_error_log` or something like that. In XAMPP, the default location of this file is `XAMPP/xamppfiles/logs/`. For other web hosting platforms, please consult the documentation or do a quick Google search (i.e. "where is the php error log in _____"). Some web hosts may provide a special interface for accessing the php error log, through ssh, cpanel, etc. Please ask them directly for help with this. |
| 30 | + |
| 31 | +- Non-fatal PHP errors will be logged in your UserFrosting error log. Look for your `app/logs/errors.log` file. |
| 32 | + |
| 33 | +- Frontend (Javascript-related) errors: in your browser's Javascript console. See [this guide](https://learn.userfrosting.com/background/client-side) to using your browser console. |
| 34 | + |
| 35 | +You should also try testing your code in a local development environment, to separate **code-related** issues from **server** issues. In general, we recommend that you install a local development server on your computer, rather than [testing your code directly on the production server](https://pbs.twimg.com/media/BxfENwpIYAAcHqQ.png). This means you can test your code directly on your own computer, making development faster and without the risk of exposing sensitive information to the public. We recommend installing [XAMPP](https://www.apachefriends.org) if you don't already have a local server set up. |
| 36 | + |
| 37 | +## Contributing to the Codebase |
| 38 | + |
| 39 | +We welcome your technical expertise! But first, please join us in [chat](https://chat.userfrosting.com) to discuss your proposed changes/fixes/enhancements before you get started. At least one member of our development team will usually be around. |
| 40 | + |
| 41 | +Please also be sure to read our [style guidelines](../STYLE-GUIDE.md). |
| 42 | + |
| 43 | +When it's time to integrate changes, our git flow more or less follows http://nvie.com/posts/a-successful-git-branching-model/. |
| 44 | + |
| 45 | +### Branches |
| 46 | + |
| 47 | +#### `master` |
| 48 | +The current release or release candidate. Always numbered as `major.minor.revision`, possibly with an `-alpha`, `-beta` or `-RC` extension as well. Commits should **never** be send directly on this branch. |
| 49 | + |
| 50 | +#### `hotfix` |
| 51 | +Contains the next bug fix release, typically matching the next `revision` version. Any changes not introducing a breaking change can be committed to this branch. Always numbered as `major.minor.revision`. |
| 52 | + |
| 53 | +When ready, changes should be merged into both **master** and **develop**. |
| 54 | + |
| 55 | +#### `develop` |
| 56 | +Contains breaking changes that will need to wait for the next version to be integrated. Typically matched the next `minor` version. Always numbered as `major.minor.x`. |
| 57 | + |
| 58 | +When ready, changes should be merged into both **master** and **hotfix**. |
| 59 | + |
| 60 | +#### `feature-*` |
| 61 | +New features that introduce some breaking changes or incomplete code should be committed in a separate `feature-{name}` branch. |
| 62 | + |
| 63 | +When ready, the branch should be **[squashed-merged](https://github.com/blog/2141-squash-your-commits)** ([guide](https://stackoverflow.com/a/5309051/445757)) into `develop` (or `hotfix` if it doesn't introduce a breaking change). |
| 64 | + |
| 65 | +### Releases |
| 66 | + |
| 67 | +After every release, the `hotfix` branch (and possibly `develop`, for minor/major releases) should immediately be version-bumped. That way, new changes can be accumulated until the next release. |
| 68 | + |
| 69 | +When a new version is created, the version number need to be changed in `app/define.php`. `CHANGELOG.md` should also be updated and the associated tag should be created on Github. |
| 70 | + |
| 71 | +#### Alpha/beta/RC releases |
| 72 | + |
| 73 | +During alpha/beta/RC, a release candidate always sits on the `master` branch. During the alpha/beta phase, major changes can still be integrated into `master` from `develop`. However, this should bump the revision number instead of the minor/major number. During RC, only _hotfixes_ can be merged into `master`. |
| 74 | + |
| 75 | +## Working together |
| 76 | + |
| 77 | +### Issues |
| 78 | + |
| 79 | +Issues are used as a todo list. Each issue represent something that needs to be fixed, added or improved. Be sure to assign issues to yourself when working on something so everyones knows this issue is taken care of. |
| 80 | + |
| 81 | +Issues are tagged to represent the feature or category it refers to. We also have some special tags to help organize issues. These includes: |
| 82 | + |
| 83 | + - [`good first issue`](https://github.com/userfrosting/UserFrosting/labels/good%20first%20issue): If this is your first time contributing to UserFrosting, look for the `good first issue` tag. It's associated with easier issues anyone can tackle. |
| 84 | + |
| 85 | + - [`up-for-grabs`](https://github.com/userfrosting/UserFrosting/labels/up-for-grabs): Theses issues have not yet been assigned to anybody. Look for theses when you want to start working on a new issue. |
| 86 | + |
| 87 | + - [`needs discussion`](https://github.com/userfrosting/UserFrosting/labels/needs%20discussion) : This issue needs to be discussed with the dev team before being implemented as more information is required, questions remain or a higher level decision needs to be made. |
| 88 | + |
| 89 | + - [`needs more info`](https://github.com/userfrosting/UserFrosting/labels/needs%20more%20info): More information is required from the author of the issue. |
| 90 | + |
| 91 | +### Milestones |
| 92 | + |
| 93 | +In order to keep a clear roadmap, milestones are used to track what is happening and what needs to be done. Milestones are used to classify problems by: |
| 94 | +- Things that need to be done ASAP |
| 95 | +- Things we are doing right now |
| 96 | +- Things we will probably do soon |
| 97 | +- Things we probably will not do soon |
| 98 | + |
| 99 | +**Things that need to be done ASAP**: this is the highest priority and this milestone should always be empty. Issues related to important bug fixes should be set on this milestone immediately. The milestone always refers to the next version of _revision_, also known as the next bugfix version. |
| 100 | + |
| 101 | +**Things we are doing right now**: this is the "main" milestone we are currently working on. Usually represents the next `minor` version, but may also represent the next major version when the focus is on the next major release. |
| 102 | + |
| 103 | +**Things we’ll probably do soon**: It's a "Next Tasks" milestone. These tasks will be addressed in the near future, but not close enough for the next version. Usually represents the second minor revision **and** the next major release. |
| 104 | + |
| 105 | +**Things we probably won’t do soon**: We refer to these issues and sometimes look through them, but they are easy to ignore and sometimes intentionally ignored. Represent issues without milestones that do not have a defined timeframe. |
| 106 | + |
| 107 | + |
| 108 | +To maintain a clear history of progress on each milestone, milestones must be closed when completed and the corresponding version released. A new milestone must then be created for the next release. In addition, the milestone version must be updated when new versions are released. |
| 109 | + |
| 110 | +## Learn documentation |
| 111 | + |
| 112 | +The [Learn Documentation](https://learn.userfrosting.com) should always be updated along side code changes. |
| 113 | + |
| 114 | +Changes to the [learn repository](https://github.com/userfrosting/learn) should follow the same logic as the main repository, ie. any changes applied to the `hotfix` branch should be documented in the learn `hotfix` branch. This also apply to `feature-*` branches. |
| 115 | + |
| 116 | +Additionally, the `learn` repository can have `dev-*` for learn specific features and fixes. |
| 117 | + |
| 118 | +## Building the API documentation |
| 119 | + |
| 120 | +To build the API documentation, install [ApiGen](http://www.apigen.org/) globally and then run: |
| 121 | + |
| 122 | +`apigen generate --source UserFrosting/app,userfrosting-assets/src,userfrosting-config/Config,userfrosting-fortress/Fortress,userfrosting-i18n/I18n,userfrosting-session/Session,userfrosting-support/Support --destination userfrosting-api --exclude *vendor*,*_meta* --template-theme "bootstrap"` |
| 123 | + |
| 124 | +from inside your dev directory. |
| 125 | + |
| 126 | +## Automatically fixing coding style with PHP-CS-Fixer |
| 127 | + |
| 128 | +[PHP-CS-Fixer](https://github.com/FriendsOfPHP/PHP-CS-Fixer) can be used to automatically fix PHP code styling. UserFrosting provides a project specific configuration file ([`.php_cs`](.php_cs)) with a set of rules reflecting our [style guidelines](../STYLE-GUIDE.md). This tool should be used before submitting any code change to assure the style guidelines are met. Every sprinkles will also be parsed by the fixer. |
| 129 | + |
| 130 | +PHP-CS-Fixer is automatically loaded by Composer and can be used from the UserFrosting root directory : |
| 131 | + |
| 132 | +``` |
| 133 | +app/vendor/bin/php-cs-fixer fix |
| 134 | +``` |
| 135 | + |
| 136 | +## Useful tools |
| 137 | + |
| 138 | +If you are using **Atom**, be sure to checkout theses useful packages : |
| 139 | + - [Docblockr](https://atom.io/packages/docblockr) : Used to generate [documentation block](https://github.com/userfrosting/UserFrosting/blob/master/STYLE-GUIDE.md#documentation). |
| 140 | + - [php-ide-serenata](https://atom.io/packages/php-ide-serenata) : Integrates [Serenata](https://gitlab.com/Serenata/Serenata) as PHP IDE, providing autocompletion, code navigation, refactoring, signature help, linting and annotations. |
0 commit comments