Problem/Motivation

Dependency spatie/data-transfer-object is not maintained anymore, see https://github.com/spatie/data-transfer-object.

Proposed resolution

Maintainer suggest migrating to spatie/laravel-data or cuyz/valinor.

Issue fork overpass_api-3331939

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

tormi created an issue. See original summary.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented

There is a blog post about this:

https://stitcher.io/blog/deprecating-spatie-dto

Orkut Murat Yılmaz opened merge request !4

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Status: Active » Needs review

I've created a Merge Request for this.

murz’s picture

murz
Primary language English
Location Yerevan, Armenia
commented
Status: Needs review » Needs work

Thank you, folks, reviewed the MR. Together with replacing the package name in the composer.json, we should also check for the API changes and replace "use" constructions in the files, and test if all works as well as previously.

Could you please do this? I'm not using this module on my website already, and not a lot of free time, so can't easily test it.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Assigned: Unassigned » orkutmuratyilmaz

orkutmuratyilmaz closed merge request !4

orkutmuratyilmaz opened merge request !5

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Status: Needs work » Needs review

@murz, I've created another MR and closed the older one.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Status: Needs review » Needs work
orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Status: Needs work » Needs review
orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Status: Needs review » Needs work

I'm checking them very soon.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented
Status: Needs work » Needs review

I've fixed those 2 files:)

murz’s picture

murz
Primary language English
Location Yerevan, Armenia
commented
Status: Needs review » Fixed

Good now! Merged.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

orkutmuratyilmaz’s picture

orkutmuratyilmaz
Primary language Turkish
Location İzmir
commented

@murz, thanks for the merge:)

murz’s picture

murz
Primary language English
Location Yerevan, Armenia
commented
Status: Fixed » Closed (fixed)