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

Added ability to ignore assets #222

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

mabentley85
Copy link

You can now set the wbraganca\dynamicform\DynamicFormAsset bundle to false when do not want to include the JavaScript file.

@SteveSimpson
Copy link

The fixes for the Javascript problems with Yii 2.0.13 helped a bunch! thanks!

@mabentley85
Copy link
Author

Thanks! Feel free to remove my name mabentley85 from everywhere, I needed to create a Composer package for my forked version so that I could use it as a dependency with a project of mine.

@SteveSimpson
Copy link

I am still having an issue with creating the 1st element in a 0 to X list. I'm not seeing where things are failing, have you seen this? ... I have a copy in my own repo, I'm doing the same thing.

@philippfrenzel
Copy link

@SteveSimpson can you pls. prepare a separate pull request with only the fixes of the js? As this seems to solve the issue #224

@SteveSimpson
Copy link

SteveSimpson commented Nov 22, 2017 via email

@spiro-stathakis
Copy link

Has @wbraganca abandoned this project? I am keen to adopt the @mabentley85 package but if this project is still live I would like to stick with it without creating a mess in managing composer. As my project is a production system I would like to keep it clean as poss.

@akula22
Copy link

akula22 commented Dec 20, 2017

What needs to be changed in the code to solve this problem?

@newerton
Copy link

+1

@sgroko
Copy link

sgroko commented Jan 22, 2021

Thank you very much for your work! You saved my work, which needs a nested solution in a dynamic view!
Stefan, with greetings from Germany to the West Coast

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants