-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
[Admin] Initial support for static content #5409
[Admin] Initial support for static content #5409
Conversation
pamil
commented
Jul 1, 2016
•
edited
Loading
edited
Q | A |
---|---|
Bug fix? | no |
New feature? | yes |
BC breaks? | no |
Related tickets | based on #5417 |
License | MIT |
1c5555e
to
4323a15
Compare
0178d80
to
eb6d8d7
Compare
/** | ||
* @Then the static content :title should not be added | ||
*/ | ||
public function theCurrencyShouldNotBeAdded($title) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Currency? :)
eb6d8d7
to
eb4395e
Compare
Thanks Kamil! |
use Webmozart\Assert\Assert; | ||
|
||
/** | ||
* @author Łukasz Chruściel <lukasz.chrusciel@lakion.com> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing to be afraid of, that's a piece of really good code! :D
Won't it be worthy to split this implementation somehow from a default admin? I can imagine many cases, where somebody would like to take our admin, but manage a static content different way. |
I guess we can port the features from the old admin panel to the new one first and then think about making it an optional feature, there's much more to do in order to achieve that. |