I've just created a patch to add Features support to Captcha.
Take a look.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | captcha-features-1311930-2.patch | 2.89 KB | scottrigby |
| captcha-features.patch | 2.03 KB | sjovanig |
I've just created a patch to add Features support to Captcha.
Take a look.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | captcha-features-1311930-2.patch | 2.89 KB | scottrigby |
| captcha-features.patch | 2.03 KB | sjovanig |
Comments
Comment #1
soxofaan commentedHi,
Cool work.
Some things:
- there is already work on exportability of the CAPTCHA module at #825088: Exportables support for CAPTCHA points
- the exporting seems to work from the superficial testing I did, but how is the importing suposed to work? I tried to override exported values and to revert them through the features UI, but that didn't seem to work
best,
soxofaan
Comment #2
scottrigby@lesergi Nice work!
@soxofaan After your commits in #825088-28: Exportables support for CAPTCHA points the above patch basically works (including importing/reverting).
But the feature would always appear overridden because of two things:
The attached patch addresses both of these issues, and works well from my testing so far.
Comment #3
wundo commented