Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Softdelete #542

Merged
merged 50 commits into from
Jul 31, 2020
Merged

Softdelete #542

merged 50 commits into from
Jul 31, 2020

Conversation

boehsermoe
Copy link
Member

@boehsermoe boehsermoe commented Jul 29, 2020

What are you changing/introducing

Tablename prefix for is_deleted column and unit tests for SoftDeleteTrait

What is the reason for changing/introducing

Problem with joinWith relations when both models use SoftDeleteTrait

QA

Q A
Is bugfix? yes
New feature? no
Breaks BC? no
Tests pass? yes

@boehsermoe boehsermoe marked this pull request as ready for review July 31, 2020 05:50
Copy link
Member

@nadar nadar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great! Thanks. Could you please add a changelog entry?

@codeclimate
Copy link

codeclimate bot commented Jul 31, 2020

Code Climate has analyzed commit dba203b and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (50% is the threshold).

This pull request will bring the total coverage in the repository to 47.5% (-0.1% change).

View more on Code Climate.

@nadar nadar merged commit 9d0a535 into luyadev:master Jul 31, 2020
@boehsermoe boehsermoe deleted the softdelete-test branch July 31, 2020 18:59
@nadar nadar mentioned this pull request Aug 4, 2020
slowfox089 pushed a commit to slowfox089/luya-module-admin that referenced this pull request Dec 10, 2020
* Block Preview styling

* Tooltip preview fix luyadev#246

* Date plugin with custom format

* Changelog for luyadev#270

* Refactored namespace using

* Connection lost handling

* Unit test fixed close luyadev#229

* CONNECTION_ID only available in mysql

* Block Preview styling

* Tooltip preview fix luyadev#246

* travis tests with php 7.4

* clean up dist

* attache find behaviours

* Changelogs

* Unit tests

* phpdocs & changelog

* Update NgRestModel.php

* Update NgRestModel.php

* Update NgRestModel.php

* Update TestNgRestModel.php

* SoftDeleteTraitTest

* SoftDeleteTraitTest

* use static tablename

* SoftDeleteTraitTest

* execute truncate

* Update CHANGELOG.md

* Update CHANGELOG.md

Co-authored-by: Basil <git@nadar.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants