Closed amandafrench closed 11 years ago
It's also highly desirable to make the difference between "Applying" (i.e., going through an approval process) vs. "Registering" (just signing up) more clear throughout. Some THATCamps do one, some do the other. The plugin is built on the logic of applying, but it mostly uses the language of registering, which confuses people. I also feel bad that we make people write that application essay when it isn't really necessary in a lot of cases. Might be nice to put that on the author page, too, or feed it into a blog post.
Can you give me more of a sense of what you want here? I don't see much in the codebase related to auto-approval. How would this work?
Here's where Jerm removed it from thatcamp-registrations-admin.php: https://github.com/chnm/thatcamp-registrations/commit/ceb3065da5bd5f216a87edd89e3ac10c669a8027 (might be in a previous version of course). The auto-approve was an admin setting that allowed site organizers to say, "When people submit the registration form, their registration will be automatically approved and they will be added to the site as an Author." It allowed the site admin to put in a "quota" number -- approve all registrations up to 75, or 100, or what have you. After that, registrations would be classified as "Pending."
It would also send the approved user the "Registration Approved" email from the THATCamp Registration Settings, of course. And the Welcome email with the username and password (or lost password link for existing users).
Ah, he removed the field because he had never written the functionality :)
This is kind of a big job, especially with the "quota" stuff. I'm going to see what other little things I can knock off before circling back on this.
I'm going to downgrade this -- it's not that important. Sorry: could've sworn this actually worked at some point, but it must just have been a dummy admin toggle.
There's currently code in the plugin that Jeremy commented out that lets the site admin auto-approve registrations up to a certain number. I thought it was working fine, but apparently it wasn't. Would like to put that option back in.