-
Notifications
You must be signed in to change notification settings - Fork 1
362 improve code #44
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
Merged
Merged
362 improve code #44
Changes from 17 commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
3a1b1da
Bring the example up to date with current application skeleton
tijsverkoyen 6f4379c
Default values for mailer
tijsverkoyen d01b1f1
Give console command a more logical name
tijsverkoyen b849e09
Use constructor property promotion
tijsverkoyen bdfe5fd
Code styling
tijsverkoyen 506b885
Fix PHPStan issues
tijsverkoyen 79c15ed
Use instead of . In prepartion of HandleTrait
tijsverkoyen 91bb92e
Fix stimulus error
tijsverkoyen f27a833
Redirect to success page
tijsverkoyen f4fab8c
Add missing translation
tijsverkoyen 6fa0f91
Use decent autocomplete attributes
tijsverkoyen 0794478
Move all #[Route] PHP attributes to class level
tijsverkoyen e677a2b
Standarised route names
tijsverkoyen 299d35e
Use symfony/test-pack
tijsverkoyen 0de0485
Use public properties instead of getters. Simplifies the code.
tijsverkoyen d52591b
Code styling
tijsverkoyen e52f794
Fix typo
tijsverkoyen 5b62984
Simplified PHPDoc for PHPStan
tijsverkoyen c644f58
Improved (added) tests
tijsverkoyen 9e85bce
Cleanup code
tijsverkoyen 5f6a335
Use dama/doctrine-test-bundle to clear the database for each test
tijsverkoyen 2974909
Test for MessageHandlers
tijsverkoyen c4085da
Use PHPUnit directly
tijsverkoyen d1c417d
Update outdated NPM packages
tijsverkoyen e855c6c
Remove useless fixtyre
tijsverkoyen 26f0427
Remove symfony/simple-phpunit
tijsverkoyen 86cefa5
Fix color attribute for PHPUnit command
tijsverkoyen b78ba11
Changes from sumocoders/application-skeleton
tijsverkoyen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| @@ -1,15 +1,16 @@ | ||||||||||||||
| assets/images/*.svg | ||||||||||||||
| assets/images/ | ||||||||||||||
| assets/vendor/ | ||||||||||||||
| bin/* | ||||||||||||||
| config/* | ||||||||||||||
| public/build/* | ||||||||||||||
| public/bundles/* | ||||||||||||||
| public/files/* | ||||||||||||||
| drivers/* | ||||||||||||||
| migrations/* | ||||||||||||||
| node_modules/* | ||||||||||||||
| public/* | ||||||||||||||
| src/* | ||||||||||||||
| tests/* | ||||||||||||||
| translations/* | ||||||||||||||
| var/* | ||||||||||||||
| vendor/* | ||||||||||||||
| **/*.dist | ||||||||||||||
| **/*.json | ||||||||||||||
| **/*.lock | ||||||||||||||
| **/*.yaml | ||||||||||||||
| **/*.yml | ||||||||||||||
| php.ini | ||||||||||||||
| *.* | ||||||||||||||
|
||||||||||||||
| *.* | |
| *.js | |
| *.html | |
| *.json | |
| *.txt | |
| *.md |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,3 @@ | ||
| { | ||
| "extends": "stylelint-config-standard" | ||
| "extends": "stylelint-config-standard-scss" | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,4 +10,4 @@ | |
| $config = new TwigCsFixer\Config\Config(); | ||
| $config->setRuleset($ruleset); | ||
|
|
||
| return $config; | ||
| return $config; | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,13 +1,10 @@ | ||
| /* | ||
| * Welcome to your app's main JavaScript file! | ||
| * | ||
| * We recommend including the built version of this JavaScript file | ||
| * (and its CSS file) in your base layout (base.html.twig). | ||
| */ | ||
| // CSS | ||
| import '@fortawesome/fontawesome-free/css/all.css' | ||
| import 'tom-select/dist/css/tom-select.default.css' | ||
| import 'tom-select/dist/css/tom-select.bootstrap5.css' | ||
| import 'flatpickr/dist/flatpickr.css' | ||
| import 'flatpickr/dist/themes/airbnb.css' | ||
|
|
||
| import { Framework } from './js/imports' | ||
|
|
||
| // start the Stimulus application | ||
| import './bootstrap'; | ||
|
|
||
| new Framework() | ||
| // JS | ||
| import 'bootstrap' | ||
| import './bootstrap.js' |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.