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

Remove yiisoft/yii-base-web #6

Closed
wants to merge 1 commit into from

Conversation

damasco
Copy link
Contributor

@damasco damasco commented Jul 11, 2019

Q A
Is bugfix?
New feature?
Breaks BC?
Tests pass?
Fixed issues

@samdark samdark added the status:to be verified Needs to be reproduced and validated. label Jul 11, 2019
@samdark samdark self-assigned this Jul 11, 2019
@samdark
Copy link
Member

samdark commented Jul 11, 2019

Technically could be done but I need to re-check it. Likely it's a better idea to move some parts of the demo to common application base.

@damasco
Copy link
Contributor Author

damasco commented Jul 11, 2019

Both of them are projects, I think you should not include one in the other.

@samdark
Copy link
Member

samdark commented Jul 11, 2019

@hiqsol I think we need you :)

@thenotsoft
Copy link
Member

I think everything is better in one package, since only one layout is planned, or is the basic and adavance layout planned?

@samdark
Copy link
Member

samdark commented Jul 12, 2019

No, there is no plan to have two application templates. The @hiqsol idea was that application template should be update-able i.e. you inherit from a template with your application.

@samdark
Copy link
Member

samdark commented Aug 20, 2019

Closing for now.

@samdark samdark closed this Aug 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status:to be verified Needs to be reproduced and validated. status:under discussion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants