-
Notifications
You must be signed in to change notification settings - Fork 1
Be/privacy modal #377
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
base: main
Are you sure you want to change the base?
Be/privacy modal #377
Conversation
| return false; | ||
| } | ||
| return true; | ||
| if (!(obj instanceof RegisteredUserDTO other)) return false; |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> reported by reviewdog 🐶
'if' construct must use '{}'s.
| return false; | ||
| } | ||
| return true; | ||
| if (!(obj instanceof RegisteredUserDTO other)) return false; |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> reported by reviewdog 🐶
'if' construct must use '{}'s.
| return true; | ||
| if (!(obj instanceof RegisteredUserDTO other)) return false; | ||
|
|
||
| return Objects.equals(id, other.id) && |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck> reported by reviewdog 🐶
'&&' should be on a new line.
| if (!(obj instanceof RegisteredUserDTO other)) return false; | ||
|
|
||
| return Objects.equals(id, other.id) && | ||
| Objects.equals(dateOfBirth, other.dateOfBirth) && |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck> reported by reviewdog 🐶
'&&' should be on a new line.
|
|
||
| return Objects.equals(id, other.id) && | ||
| Objects.equals(dateOfBirth, other.dateOfBirth) && | ||
| Objects.equals(email, other.email) && |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck> reported by reviewdog 🐶
'&&' should be on a new line.
| Objects.equals(schoolOther, other.schoolOther) && | ||
| Objects.equals(teacherPending, other.teacherPending) && | ||
| Objects.equals(lastSeen, other.lastSeen) && | ||
| Objects.equals(registeredContexts, other.registeredContexts) && |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.whitespace.OperatorWrapCheck> reported by reviewdog 🐶
'&&' should be on a new line.
| return false; | ||
| } | ||
| return true; | ||
| if (!(obj instanceof RegisteredUser other)) return false; |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.blocks.NeedBracesCheck> reported by reviewdog 🐶
'if' construct must use '{}'s.
| import uk.ac.cam.cl.dtg.isaac.api.GameboardsFacade; | ||
| import uk.ac.cam.cl.dtg.isaac.api.managers.GameManager; | ||
| import uk.ac.cam.cl.dtg.isaac.api.managers.NoWildcardException; | ||
| import uk.ac.cam.cl.dtg.isaac.api.exceptions.NoWildcardException; |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> reported by reviewdog 🐶
Wrong lexicographical order for 'uk.ac.cam.cl.dtg.isaac.api.exceptions.NoWildcardException' import. Should be before 'uk.ac.cam.cl.dtg.isaac.api.managers.GameManager'.
| import uk.ac.cam.cl.dtg.isaac.api.managers.EventBookingManager; | ||
| import uk.ac.cam.cl.dtg.isaac.api.managers.EventBookingUpdateException; | ||
| import uk.ac.cam.cl.dtg.isaac.api.managers.EventIsCancelledException; | ||
| import uk.ac.cam.cl.dtg.isaac.api.exceptions.EventBookingUpdateException; |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> reported by reviewdog 🐶
Wrong lexicographical order for 'uk.ac.cam.cl.dtg.isaac.api.exceptions.EventBookingUpdateException' import. Should be before 'uk.ac.cam.cl.dtg.isaac.api.managers.EventBookingManager'.
| import uk.ac.cam.cl.dtg.isaac.api.managers.EventBookingUpdateException; | ||
| import uk.ac.cam.cl.dtg.isaac.api.managers.EventIsCancelledException; | ||
| import uk.ac.cam.cl.dtg.isaac.api.exceptions.EventBookingUpdateException; | ||
| import uk.ac.cam.cl.dtg.isaac.api.exceptions.EventIsCancelledException; |
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.
🚫 [checkstyle] <com.puppycrawl.tools.checkstyle.checks.imports.CustomImportOrderCheck> reported by reviewdog 🐶
Wrong lexicographical order for 'uk.ac.cam.cl.dtg.isaac.api.exceptions.EventIsCancelledException' import. Should be before 'uk.ac.cam.cl.dtg.isaac.api.managers.EventBookingManager'.
|


No description provided.