search

ThomasTJdev / nim_websitecreator

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

Updated db_connector and otp package changes. #145

Closed prasad83 closed 10 months ago

prasad83 commented 11 months ago

db_connector/ prefix should be use for (db_common, db_sqlite, db_postgres) $newTotp... to be changed to $(Totp.init...

ThomasTJdev commented 11 months ago

Great PR @prasad83 !

Would you change the import of the db_* to have backward compatibility like:

when NimMajor >= 2:
  import
    db_connector/db_postgres
else:
  import
    std/db_postgres
prasad83 commented 11 months ago

@ThomasTJdev - thank you for the hint. I have now added backward compat fixes.