-
Notifications
You must be signed in to change notification settings - Fork 21
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
Automate submissions #70
Comments
On site screenshots – at the moment the site's layouts only work if the image is an annoying 1440x1200 (those exact numbers or that ratio, unsure). If we do expect third-party users to upload their own images, we should make sure that:
See #31 (comment) for an analysis of screenshot size requirements. |
So users need:
As for images size, surely we can setup the |
From @johnanders123's input over Slack, we will also need:
Edit: IP whitelist is easy to disable Edit: I intend to trial this whole user registration flow manually and see what happens. |
Here is the permission set up I made manually:
It doesn't take me much more time to do this once than to set up the usual pages, so I'll look for volunteers to try this out. |
A solution is currently being discussed and estimated at Springload, and will hopefully be resourced shortly. |
@mojeto - please speak to @loicteixeira about the current state of the solution for this one. |
At a high level, the work will happen in steps:
See the full proposal and estimate for more background and details. |
Current WIP solution:
Pros:
Cons:
Solution discussed offline with @tacitus
Pros:
Cons:
|
django-allauth can't be used together with Wagtail till the clash between Wagtail Site and Django Site resolved. Some work is being done, but it's a blocker currently. |
As discussed offline, we should not forget that we will need to create accounts for the developers pages which already exist so we might need to keep the email login for them. |
Thanks I have just run into this clash as I added allauth to my project. |
Edit @loicteixeira:
The actual proposal (scope) is linked below.
Rough plan:
The text was updated successfully, but these errors were encountered: