search

pepijn / omni_kassa

Easier Rabobank OmniKassa payments
MIT License
9 stars 7 forks source link

Add key version to configuration #4

Closed joshuajansen closed 10 years ago

joshuajansen commented 10 years ago

Hi!

I kept getting this error: screenshot 2013-12-19 15 57 44

Eventually I found that it had to do with the key_version which was 2 in my case but defaults to 1 in your gem. Ofcourse it is possible to set the key_version in my controller like

omnikassa = OmniKassa::Request.new
omnikassa.key_version = 2

But I think it would be nicer to encourage users to verify that their key_version is indeed 1 by letting them specify it in their initializer. ( I know it would've saved me some time :wink: )

Thanks for making this gem!

pepijn commented 10 years ago

Thank you! :-)

Can you tell me why the version changed to 2?

joshuajansen commented 10 years ago

I had to generate a new key, therefore the key_version incremented: screenshot 2013-12-20 08 38 42

pepijn commented 10 years ago

Oh wow, I didn't even know the key_version was meant for that...

pepijn commented 10 years ago

The gem has been updated!

joshuajansen commented 10 years ago

Thanks! :smile: