mpociot / versionable

Laravel Model versioning made easy
MIT License
757 stars 101 forks source link

Refactor Encoders and add JSON support #93

Open erikgaal opened 2 years ago

erikgaal commented 2 years ago

Builds upon #83 and refactors it a bit to provide a clear solution. Also rebased the PR so it's up to date with the trunk.

Thanks @lsmith77 for the original PR!

nonoesp commented 2 years ago

Hi, @lsmith77 and @erikgaal I don't have the bandwidth to take on new features and maintain this project.

Would any of you want to help?

These pull requests are a really good start; it's just that I haven't even had time to figure out whether there are parts that break with your changes.

CoolGoose commented 4 weeks ago

@nonoesp since we're starting to use this more in our company, is there something I can start helping with to review / add tests so we can get some of these things moving forward ? Json support is something we're going to need, and I think it would be nice to do it part of this and not a fork.

nonoesp commented 4 weeks ago

@CoolGoose Hey! I have limited bandwidth for this. But I'll be happy to chat and let you maintain this package if you'd like. =)

CoolGoose commented 4 weeks ago

I can give it a try since I should be able to have a few hours per week for this. Let me review this PR first, and see how this goes ;)