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.

Sign up for GitHub

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

Jump to bottom

Provide wrapper on top of WTForms #10

Open
porduna opened this issue Dec 12, 2012 · 2 comments
Open

Provide wrapper on top of WTForms #10

porduna opened this issue Dec 12, 2012 · 2 comments
Labels
rlms plugins

Comments

@porduna
Copy link
Member

porduna commented Dec 12, 2012

The RLMS API right now is highly coupled to WTForms. A change that breaks compatibility in WTForms would invalidate all the RLMS plug-ins. Given that the source code is easily wrappable, should we create a form generator which basically generates WTForms classes so as to decouple it or should we just decide to be fine with this coupling? I 👍 for either option.
@nhocki
Copy link
Member

nhocki commented Dec 13, 2012

Decouple something is better IMO. +1 for the change.

@porduna
Copy link
Member Author

porduna commented Dec 13, 2012

Ok. I'd suggest starting with a very simple one and increasing it as more RLMSs are added. Anyway #4 goes first

@nhocki nhocki mentioned this issue Mar 25, 2013
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
rlms plugins
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nhocki @porduna