-
Notifications
You must be signed in to change notification settings - Fork 32
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
Changed the initial comic format load to a request/response [#642] #664
Conversation
@@ -35,20 +39,19 @@ | |||
@RestController | |||
@Log4j2 | |||
public class ComicFormatController { | |||
private static final String LOAD_COMIC_FORMATS = "load.comic-formats"; | |||
private static final String ADD_COMIC_FORMAT_QUEUE = "/queue/comic-format.add"; | |||
|
|||
@Autowired private ComicFormatService comicFormatService; | |||
@Autowired private SimpMessagingTemplate messagingTemplate; | |||
|
|||
/** Retrieves the list of all comic formats and publishes them. */ | |||
@MessageMapping(LOAD_COMIC_FORMATS) |
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.
Need to add the LOAD_COMIC_FORMATS constant in Constants interface
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.
@BRUCELLA2 I missed committing some local changes before pushing the PR. They're up now.
ad41f6f
to
c67eb46
Compare
c67eb46
to
7a944f5
Compare
Kudos, SonarCloud Quality Gate passed! |
This cuts the overhead of the formats being broadcast to every user.
Status
READY
Does this PR contain migrations?
NO
Before You Submit Your PR: