petrbroz / forge-sdk-python

Unofficial Autodesk Forge SDK for Python.

Home Page:https://petrbroz.github.io/forge-sdk-python/index.html

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Wrong type annotation

paoloq opened this issue · comments

Hi, I encountered an error caused by a wrong type annotation.

def get_authorization_url(
        client_id: str, response_type: str, redirect_uri: str,
        scopes: list[Scope], state: str = None
    ) -> str

As you can see in the case above, list[Scope] causes a runtime error because list is not the right way to type annotate.
It should be changed in List, imported from typing.

I'm going to PR a fix in few minutes, I created this issue just for completeness.

Btw, thanks for starting this great work in creating an SDK for Forge API.
I'm likely to start contributing in next weeks as I'll need it for a new project.

Hi @paoloq
I think this is related to #9.
If you use python 3.9 or later you should not get the error.

I've updated the typings in the latest commit (432b3ce). To be honest, I'm not very experienced with typings in Python, so I'm not sure whether to use typings.List or list?