Skip to content

[BUGFIX] BaseController.Shutdown is now correctly removing events#43

Merged
loic-lopez merged 1 commit into
masterfrom
bugfix/mvc-shutdown
Aug 24, 2020
Merged

[BUGFIX] BaseController.Shutdown is now correctly removing events#43
loic-lopez merged 1 commit into
masterfrom
bugfix/mvc-shutdown

Conversation

@loic-lopez

Copy link
Copy Markdown
Owner

No description provided.

@loic-lopez loic-lopez self-assigned this Aug 24, 2020
@loic-lopez loic-lopez added bug Something isn't working UMVC.Core UMVC.Core project's issue labels Aug 24, 2020
@codecov

codecov Bot commented Aug 24, 2020

Copy link
Copy Markdown

Codecov Report

Merging #43 into master will not change coverage.
The diff coverage is 0.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #43   +/-   ##
=======================================
  Coverage   69.95%   69.95%           
=======================================
  Files          44       44           
  Lines        1175     1175           
  Branches       19       19           
=======================================
  Hits          822      822           
  Misses        342      342           
  Partials       11       11           
Impacted Files Coverage Δ
UMVC.Core/MVC/BaseController.cs 75.86% <0.00%> (ø)

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 b457cd0...e431b03. Read the comment docs.

@loic-lopez loic-lopez changed the title [BUGFIX] BaseController.Shutdown is removing correctly events [BUGFIX] BaseController.Shutdown is now correctly removing events Aug 24, 2020
@loic-lopez loic-lopez merged commit 6ff59ac into master Aug 24, 2020
@loic-lopez loic-lopez deleted the bugfix/mvc-shutdown branch August 24, 2020 09:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working UMVC.Core UMVC.Core project's issue

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant