Skip to content

Feature/disable model save signals#285

Merged
auvipy merged 3 commits intojazzband:masterfrom
ebsaral:feature/disable-model-save-signals
May 20, 2019
Merged

Feature/disable model save signals#285
auvipy merged 3 commits intojazzband:masterfrom
ebsaral:feature/disable-model-save-signals

Conversation

@ebsaral
Copy link
Copy Markdown
Contributor

@ebsaral ebsaral commented Jul 10, 2017

Problem

When saving an object, I want to disable pre_save/post_save signals.

Solution

I overwrote the save method of the base model class.

Commandments

  • Write PEP8 compliant code.
  • Cover it with tests.
  • Update CHANGES.rst file to describe the changes, and quote according issue with GH-<issue_number>.
  • Pay attention to backward compatibility, or if it breaks it, explain why.
  • Update documentation (if relevant).

Copy link
Copy Markdown
Contributor

@auvipy auvipy left a comment

Choose a reason for hiding this comment

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

please fix merge conflicts

@MRigal
Copy link
Copy Markdown
Member

MRigal commented May 16, 2019

@ebsaral any chance to fix the merge conflicts?

@ebsaral
Copy link
Copy Markdown
Contributor Author

ebsaral commented May 16, 2019

@ebsaral any chance to fix the merge conflicts?

Ah, sorry. I will do it today.

Peace.

@ebsaral ebsaral force-pushed the feature/disable-model-save-signals branch from 2ce55f2 to b99307a Compare May 20, 2019 17:35
@ebsaral
Copy link
Copy Markdown
Contributor Author

ebsaral commented May 20, 2019

@auvipy Fixed merge conflicts.

@codecov
Copy link
Copy Markdown

codecov bot commented May 20, 2019

Codecov Report

Merging #285 into master will increase coverage by 0.15%.
The diff coverage is 96.29%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #285      +/-   ##
==========================================
+ Coverage   91.56%   91.72%   +0.15%     
==========================================
  Files           6        6              
  Lines         735      761      +26     
==========================================
+ Hits          673      698      +25     
- Misses         62       63       +1
Impacted Files Coverage Δ
model_utils/models.py 97.72% <96.29%> (-0.66%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 25dde41...f5d4e5c. Read the comment docs.

@ebsaral ebsaral force-pushed the feature/disable-model-save-signals branch from b99307a to f5d4e5c Compare May 20, 2019 17:46
@adamchainz adamchainz mentioned this pull request Oct 26, 2023
5 tasks
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.

3 participants