Skip to content

Conversation

@Hardelele
Copy link
Member

@Hardelele Hardelele commented Dec 19, 2020

@Hardelele Hardelele linked an issue Dec 19, 2020 that may be closed by this pull request
@mamadaliev
Copy link
Contributor

Exception handler shouldn't be in the library

@@ -1,7 +1,7 @@
package com.reckue.post.controller;

import com.reckue.libs.exception.ReckueUnauthorizedException;
Copy link
Contributor

Choose a reason for hiding this comment

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

Change the ReckueException to BaseException.
Because the name of the service may change in the future.

package com.reckue.post.exception.model.comment;

import com.reckue.post.exception.ModelNotFoundException;
import com.reckue.libs.exception.ReckueException;
Copy link
Contributor

Choose a reason for hiding this comment

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

let's remove the plural everywhere

@mamadaliev
Copy link
Contributor

@Hardelele
Fix the build before take away the WIP status.

image

@mamadaliev mamadaliev changed the title Move base exceptions to exception libs WIP Move base exceptions to exception libs Jan 6, 2021
@mamadaliev mamadaliev added the example Example of the new feature label Jan 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

example Example of the new feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move exceptions to exception libs

4 participants