-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
bug: Fixed notification when the user or project is deleted #4920
base: master
Are you sure you want to change the base?
Conversation
def name | ||
"Deleted User" | ||
end | ||
end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
end
at 7, 2 is not aligned with class
at 3, 0.
end | ||
end | ||
|
||
class DeletedProject |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent indentation detected.
# app/models/null_objects.rb | ||
|
||
class DeletedUser | ||
def name |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use 2 (not 4) spaces for indentation.
"Deleted User" | ||
end | ||
end | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing whitespace detected.
"Deleted Project" | ||
end | ||
end | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Trailing whitespace detected.
@@ -0,0 +1,14 @@ | |||
# app/models/null_objects.rb |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Missing frozen string literal comment.
Code Climate has analyzed commit f946c95 and detected 6 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
Fixes #
Describe the changes you have made in this PR -
I deleted the old part of code and implemented the null object pattern in ForkNotification class.
I added a file (deleted_objects.rb) where i defined the null objects: deleted user and deleted project.
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.