search

ThomasTJdev / nim_websitecreator

Nim fullstack website framework - deploy a website within minutes
https://nimwc.org
MIT License
176 stars 7 forks source link

Require newer `recaptcha` #130

Closed narimiran closed 4 years ago

narimiran commented 4 years ago

Currently in your nimwc.nimble you have requires "recaptcha >= 1.0.2", and nimble seems to pick up recaptcha at exactly that version. The problem lies with the fact that recaptcha v1.0.2 doesn't work with the latest Nim (see https://github.com/euantorano/recaptcha.nim/pull/2).

The solution for this is bumping the required version: requires "recaptcha >= 1.0.3"

ThomasTJdev commented 4 years ago

Thanks @narimiran - good catch. I made a PR at recaptcha 12 days ago, so I had the updated version while testing. I'll update nimwc.nimble and publish a new release.

Thoughts: Maybe we should have a test, which changes the requirements from >= to ==. Maybe a Nimble-package...