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

Invite Users - Not working for main admin user #57

Open
slibby opened this issue Dec 22, 2014 · 4 comments
Open

Invite Users - Not working for main admin user #57

slibby opened this issue Dec 22, 2014 · 4 comments

Comments

@slibby
Copy link

slibby commented Dec 22, 2014

When inviting users as the unmeerim user, the invite process goes smoothly, but when the new user attempts to create an account they receive a message saying "Registrations are not open at this time"

STR:

@jj0hns0n
Copy link
Contributor

The root cause of this is that REGISTRATION_OPEN was false and the invite function requires that its open. It should obviously not offer this invite option if it doesnt work.

I've temporarily opened registration and the invite feature now works.

From here we can do one of 3 thing

  • Leave registration open as is
  • turn the registration back off and hide the invite option
  • Send a patch against our fork of django-user-accounts to let the invite function work when registration is closed

@state-hiu @vdeparday @ingenieroariel @simod

@state-hiu
Copy link
Contributor

Can we leave registration open, but hide it from the UI?

@jj0hns0n
Copy link
Contributor

Thats also a possibility. You would just have to know the URL to register.

Ill just do that now and we can discuss a better longer term strategy.

On Mon, Dec 22, 2014 at 10:21 AM, Humanitarian Information Unit <
[email protected]> wrote:

Can we leave registration open, but hide it from the UI?


Reply to this email directly or view it on GitHub
#57 (comment)
.

@state-hiu
Copy link
Contributor

I think a forking/refractoring of the invite user process and of https://github.com/pinax/django-user-accounts package at the code sprint is a good idea. There's probably other minor UI/model changes that could help out with this process.

Related: GeoNode/geonode#1856

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

No branches or pull requests

2 participants