masroorejaz / react-simple-captcha

A very simple and powerful captcha for ReactJS
40 stars 19 forks source link

[FIX Spelling] Backwards compatible renaming #22

Closed thomas-br closed 1 year ago

thomas-br commented 1 year ago

@masroorejaz Adding onto the discussions from #13 / #8, can't we do a backwards compatible renaming? So correcting but keep exporting the misspelled variant?

masroorejaz commented 1 year ago

Thomas my buddy Thank you for your contribution but problems will still remain. If somebody is using the current version and he/she will deploy it on somewhere and In case I merge your changes than all of the sudden the project will stop working due to spelling change.

Therefore closing without merging.

Thanks again and keep up the good work :)

Following you on Github!

thomas-br commented 1 year ago

@masroorejaz I don't think it will break as both spellings are being exported. The correction is backwards compatible.

Existing consumers can continue using the wrong spelling, and at the same time you can use the corrected one.

masroorejaz commented 1 year ago

FIX Spelling  Backwards compatible renaming by thomas-br · Pull Request #22 · masroorejaz_react-sim I maybe missing something here but in that case you need to create function with two spellings. The changes you have made only have function with one spelling. It will not be backward compatible in that case.

thomas-br commented 1 year ago

I maybe missing something here but in that case you need to create function with two spellings. The changes you have made only have function with one spelling. It will not be backward compatible in that case.

The screenshot of yours is just the README, there I replace – correct.

But in the class itself, I exported both variants:

https://github.com/thomas-br/react-simple-captcha/blob/320ea40891d7e3425d66cbd05a119516a4f7b898/src/hooks/react-simple-captcha.js#L13

image

masroorejaz commented 1 year ago

Oh yeah didn't saw that.

I am kind of really busy right now. Will merge and release a new version once I have got free.

Thank you for your contribution.

masroorejaz commented 1 year ago

It's still causing bugs. Best to leave it that way.