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

Improved Add your site #1

Closed
wants to merge 9 commits into
base: master
from

Conversation

Projects
None yet
2 participants
@angerangel

angerangel commented Dec 17, 2013

I don't know how your server works, but I suppose that these two files should work... ;-)
Let me know

@rgchris

This comment has been minimized.

Show comment
Hide comment
@rgchris

rgchris Dec 17, 2013

Contributor

Thanks @angerangel, though the site is generated dynamically, the rebsite.r index is part of the repository, so any additions would have to be merged back—unless I were to cast it loose.

Also, on this site, RSP files aren't evaluated from the web root—I'd construct a controller for brokering the add-site method: show the blank form, verify submitted data, then either reshow the form with errors, or update the rebsite.r index and redirect: Example—of course you can do more verification and validation this way, but QM gives you the dialect and functions to flow data through in this way...

Contributor

rgchris commented Dec 17, 2013

Thanks @angerangel, though the site is generated dynamically, the rebsite.r index is part of the repository, so any additions would have to be merged back—unless I were to cast it loose.

Also, on this site, RSP files aren't evaluated from the web root—I'd construct a controller for brokering the add-site method: show the blank form, verify submitted data, then either reshow the form with errors, or update the rebsite.r index and redirect: Example—of course you can do more verification and validation this way, but QM gives you the dialect and functions to flow data through in this way...

@rgchris rgchris closed this Dec 17, 2013

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