pydantic.error_wrappers.ValidationError: 6 validation errors for ParsingModel[PaginatedFileList] about merge-python-client HOT 11 CLOSED

You can expect a fix by EoD today. Will update in this thread when its pushed out.

from merge-python-client.

@dsinghvi

You should change the type to typing.Optional[typing.List[typing.Dict[str, typing.Any]]]

If it's okay, I'll fork it and open a new PR addressing this issue

from merge-python-client.

Got it -- that makes sense, the root cause is that the OpenAPI spec is incorrectly representing the API. So @rmkonnur is planning to fix the OpenAPI generation which will then regenerate a proper client library.

from merge-python-client.

@rmkonnur

from merge-python-client.

When can I expect the fix?

from merge-python-client.

Should I create our own version of merge client that sends request directly to the API endpoint? I personally believe that not able to list files is a P1 issue and should be fixed ASAP. This has been blocking the service for few days now.

from merge-python-client.

@hwuiwon one option to unblock yourself is to make this change to the library locally (#39). Can you make that change and confirm that the SDK works end-to-end?

Apologize for any inconvenience.

from merge-python-client.

@dsinghvi I just tried and that would yield a different error.

12 validation errors for ParsingModel[PaginatedFileList]
__root__ -> results -> 0 -> permissions
  str type expected (type=type_error.str)
__root__ -> results -> 0 -> permissions -> 0
  str type expected (type=type_error.str)
__root__ -> results -> 1 -> permissions
  str type expected (type=type_error.str)
__root__ -> results -> 1 -> permissions -> 0
  str type expected (type=type_error.str)
__root__ -> results -> 2 -> permissions
  str type expected (type=type_error.str)
__root__ -> results -> 2 -> permissions -> 0
  str type expected (type=type_error.str)
__root__ -> results -> 3 -> permissions
  str type expected (type=type_error.str)
__root__ -> results -> 3 -> permissions -> 0
  str type expected (type=type_error.str)
__root__ -> results -> 4 -> permissions
  str type expected (type=type_error.str)
__root__ -> results -> 4 -> permissions -> 0
  str type expected (type=type_error.str)
__root__ -> results -> 5 -> permissions
  str type expected (type=type_error.str)
__root__ -> results -> 5 -> permissions -> 0
  str type expected (type=type_error.str)

from merge-python-client.

@hwuiwon any chance you are around for 15 min to hop on a zoom? You can join the fern discord and we can chat more realtime there.

from merge-python-client.

Im encountering this problem. It seems that previously the permissions field was a list of (str) ids, like:

  "permissions": [
    "1496fd39-1b31-42be-9aa4",
    "99b321d0-5dad-4268-b73c",
    ]

But now the permission object looks like:

So i guess just changing the type to str | JSON would fix the library.. but its a kind of breaking change

from merge-python-client.

Hi @hwuiwon, I pushed an update of the File Storage OpenAPI spec to generate the SDK this morning. Please try it out to confirm it fixes your issue. Apologies in the delay here - feel free to escalate via Intercom in the bottom right hand corner of the Merge dashboard for higher priority issues.
Thanks

from merge-python-client.