Skip to content

simple signup form with mongodb recipe #438

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

Merged
merged 2 commits into from
Jun 28, 2011
Merged

simple signup form with mongodb recipe #438

merged 2 commits into from
Jun 28, 2011

Conversation

fspillner
Copy link
Contributor

Re-added to keep the changes clean. I did the changes what already discussed at following pull request:

#433

this :doc:`file</cookbook/doctrine/mongodb>` recipe first to learn
how to setup the MongoDB Bundle to be able to work with MongoDB.

The simple Account model
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So, if a developer is creating a registration form, then they'll likely want these users to also be able to login with their new account. In other words, they'll likely need to integrate this Account entity with the security system. So, I'm wondering if we should rename Account to User (since that's what you most commonly see with security) and also make it implement the UserInterface, just since this will be a common requirement. You could just mention that this entry isn't concerned with security, but how the User class is already setup to be easily integrated with the security system.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, you're right. I added a note about this and changed the class name.

@weaverryan weaverryan merged commit 2ce7e78 into symfony:master Jun 28, 2011
@weaverryan
Copy link
Member

Looks great - merged in, thanks!

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

Successfully merging this pull request may close these issues.

2 participants