Skip to content

origin_visit_add: Change return type to OriginVisit

Antoine R. Dumont requested to merge generated-differential-D2821-source into generated-differential-D2821-target

What's really used after that call is the visit object and not the origin one.

Before:

     def origin_visit_add(
          self, origin: Origin,
          date: Union[datetime.datetime, str],
          type: str) -> Optional[Dict[str, BaseModel]]:

After:

    def origin_visit_add(
            self, origin: Origin,
            date: Union[str, datetime.datetime],
            type: str) -> OriginVisit:

Might as well remove the indirection

Depends on D2820

Test Plan

tox

Migrated from D2821 (view on Phabricator)

Merge request reports