-
Notifications
You must be signed in to change notification settings - Fork 3
Multisites v2 design #16
Comments
I've never used multisites, so I can't really comment on it. But from what I understand, this seems it be a valuable update and a future prove concept. 👍 |
@Guite What checks occur via IP address that you mentioned? Would such checks restrict functionality if a site is on server cluster and load balancer that may have varied IP addresses? |
@shefik the validator is contained in Symfony: http://symfony.com/doc/current/reference/constraints/Ip.html |
@Guite Okay, so this is only just validating that the input is a real IP Address? |
Yes. |
Here is an updated model version: Most important changes include:
|
Todo: add upload fields on the |
Probably refs zikula/core#966. |
Next update reflecting the most current design iteration: Recent changes:
|
Design iterations are completed for v2, therefore this ticket can be closed. |
Nice job 👍 |
Gabriel (@gfr) and me are going to do a new major version of Multisites.
Today I created a first model just reflecting the current state of the module plus some minor additions, like some custom actions and ip address checks and an index for the site table. I think that the model will be subject of several iterations when we incorporate open tickets and wishes into the concept.
Controller:
![multisites_v3_controller](https://cloud.githubusercontent.com/assets/277531/3360216/970bb75c-faf0-11e3-95fe-ece21538328d.png)
Model:
![multisites_v3_model](https://cloud.githubusercontent.com/assets/277531/3360217/97106590-faf0-11e3-96da-7b5d1328b03c.png)
cc: @Drak @rallek @cmfcmf
The text was updated successfully, but these errors were encountered: