[
https://issues.apache.org/jira/browse/WAVE-313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13175494#comment-13175494
]
[email protected] commented on WAVE-313:
----------------------------------------------------
bq. On 2011-12-23 04:04:28, Lennard de Rijk wrote:
bq. >
/src/org/waveprotocol/box/server/robots/agent/registration/RegistrationRobot.java,
line 2
bq. > <https://reviews.apache.org/r/3296/diff/1/?file=65765#file65765line2>
bq. >
bq. > Should this really still say Google Inc?
Probably not. What should be there instead? Or just leave out that line?
I would copy the rest of the code base, but that all seems to still reference
Google.
bq. On 2011-12-23 04:04:28, Lennard de Rijk wrote:
bq. > /src/org/waveprotocol/box/server/robots/agent/RobotAgentUtil.java, line
98
bq. > <https://reviews.apache.org/r/3296/diff/1/?file=65764#file65764line98>
bq. >
bq. > Is there a way to factor welcomebot in to this?
Not directly into the createUser method (if it lives in RobotAgentUtil.java)
without creating circular imports.
Though, when WAVE-314 is implemented, this would be solved too.
- Ali
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3296/#review4096
-----------------------------------------------------------
On 2011-12-22 23:52:27, Ali Lown wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/3296/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-12-22 23:52:27)
bq.
bq.
bq. Review request for wave.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. Implements suggestions in WAVE-313:
bq. - Adds a hideRegistration option to the server config file
bq. - Removes the auth/registration route when this is the case
bq. - Adds a RegistrationRobot to let the admin user always create new users
bq.
bq. Know limitations/features:
bq. - Doesn't remove any 'register' links - server 404s when they are clicked.
bq. - Manually created users don't get WelcomeBot'd
bq.
bq.
bq. This addresses bug WAVE-313.
bq. https://issues.apache.org/jira/browse/WAVE-313
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. /server-config.xml 1213039
bq. /server.config.example 1213039
bq. /src/org/waveprotocol/box/server/CoreSettings.java 1213039
bq. /src/org/waveprotocol/box/server/ServerMain.java 1213039
bq. /src/org/waveprotocol/box/server/robots/agent/RobotAgentUtil.java
1213039
bq.
/src/org/waveprotocol/box/server/robots/agent/registration/RegistrationRobot.java
PRE-CREATION
bq.
bq. Diff: https://reviews.apache.org/r/3296/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq. Compiled + run. Tried to access auth/register with hideRegistration=false
and hideRegistration=true.
bq. Logged in as the admin user and tried to create new accounts with the
RegistrationRobot - then logged into new accounts to check valid.
bq.
bq.
bq. Thanks,
bq.
bq. Ali
bq.
bq.
> option to disable "register new user" on web frontend, possibility for
> admin-user to add manually new users
> -----------------------------------------------------------------------------------------------------------
>
> Key: WAVE-313
> URL: https://issues.apache.org/jira/browse/WAVE-313
> Project: Wave
> Issue Type: New Feature
> Components: Web Client
> Reporter: JGB
> Assignee: Ali Lown
> Priority: Minor
> Labels: features
>
> As today, when you set up a new Wave-in-a-box-Server, everyone who has access
> to the web Client can create a new user. I think, there are a lot of people
> who would like to create private Wave-Server, so it would be nice to have an
> option to disable the "register new users"-site. As a consequence of this,
> there should also be the possibility to register as admin-user manually new
> accounts.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
