Feature: expose `timezone` and `use_tz` params in RegisterTortoise class about tortoise-orm HOT 5 CLOSED

Currently, you can use the config argument:

@asynccontextmanager
async def lifespan(app: FastAPI):
    async with RegisterTortoise(
        app,
        config={
            "connections": {"default": DB_URL},
            "apps": {"models": {"models": ["models"]}},
            "use_tz": True,
            "timezone": "Asia/Shanghai",
            "generate_schemas": True,
        },
    ):
        yield

from tortoise-orm.

Can someone answer this question? Or point out if there is something unreasonable?

from tortoise-orm.

Hi!

I see no problem with exposing timezone params in RegisterTortoise
Not sure why it wasn't added in the first place, probably contributor just forgot about it

You can contribute by making PR with this change

and I surely want to know If the db_url and config and config_file is identical each other?

Don't really understand issue here, this code just makes sure that you use only one of init methods, as we wouldn't know how to parse it if several are provided at the same time

from tortoise-orm.

Don't really understand issue here, this code just makes sure that you use only one of init methods, as we wouldn't know how to parse it if several are provided at the same time

My bad, I just don't know why adding timezone to db_url is causing an error, and I'm confused about whether all three are the same.

You can contribute by making PR with this change

Alright, I will create a new pull request for adding time_zone and use_tz in RegisterTortoise .

from tortoise-orm.

The feature has been extended in pr #1649

from tortoise-orm.