from api_analytics.fastapi import Analytics fails about api-analytics HOT 7 CLOSED

Hi thanks for raising this issue. This is due to Python version incompatibility. The style of type hints used in the code is only supported by python >= 3.9. I will revert to using the older and the more backwards compatible typing hint system and create a new release that should solve your issue on Python 3.8.

from api-analytics.

Upgrading your api-analytics package to 1.1.3 should solve this issue now. Thanks

from api-analytics.

Thanks for the quick response Tom.

Not sure if other users of this service have reported this or not, but add_middleware() function does not take 2 positional arguments (latest version of fastapi (0.95.1) as well as 0.92.1) anymore. So, I am struggling with the following line from fastapi-analytics.

app.add_middleware(Analytics, <API-KEY>) # Add middleware

Similar call to add_middleware for CORS support uses the following approach and it works:

app.add_middleware(
    CORSMiddleware,
    allow_origins=['*'],
    allow_credentials=True,
    allow_methods=["*"],
    allow_headers=["*"],
)

from api-analytics.

Oh sorry about that, I think the docs are slightly wrong there. The API key should be passed as a named argument. I'll fix that.

So it should be:
app.add_middleware(Analytics, api_key=<API-KEY>)

from api-analytics.

Thanks @tom-draper . That took care of the exception - I see that you already updated the documentation!

Made a few API calls (about 10 calls across 3 different API end-points on this API), but it doesn't show in the dashboard. Is there a long lag? Or is there some other issue? The documentation suggests making this middleware call and then directly access the dashboard. Anything that I am missing?

from api-analytics.

Apologies again! Should be fixed now. Lots of backend upgrades have been made over the past week so there are still a few bugs to iron out. Thanks a lot

from api-analytics.

Works now. Thanks

from api-analytics.