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

Add newsletter signup example #6

Merged
merged 4 commits into from
May 9, 2021

Conversation

hidalgopl
Copy link
Contributor

@hidalgopl hidalgopl commented Apr 25, 2021

Hey @zodman @jonathan-s
I just tried sockpuppet and wanted to understand how I can effectively use forms to persist a state in DB, so I added this simple newsletter signup example which covers Create/Read/Update/Delete operation using reflexes.

Please let me know if anything here could be done better / simpler, as I'm new sockpuppet user :) Thanks!

@zodman
Copy link
Owner

zodman commented Apr 27, 2021

@hidalgopl simple example thanks!

For accept your PR plz use the existing template structure and add the link to the menu. without it no body will see you your example.

@hidalgopl
Copy link
Contributor Author

thanks @zodman ! Edited templates as you suggested 👍

class CalendarMixin(MixinBase):
files = (
('core/views/calendar.py', 'python', 'python3'),
# ('core/reflexes/calendar_reflex.py', 'python', 'python3'),
# ('core/javascript/controllers/calendar_controller.js', 'javascript', 'javascript'),
# ('core/reflexes/calendar_reflex.py', 'python', 'python3'),
Copy link
Owner

Choose a reason for hiding this comment

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

remove not used code plz

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍 removed

@zodman zodman merged commit 6885187 into zodman:master May 9, 2021
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