Skip to content
This repository has been archived by the owner on Jul 13, 2023. It is now read-only.

Remove merge comment, clarify wording, and drop useless return value #1026

Closed
jrconlin opened this issue Sep 26, 2017 · 0 comments
Closed

Remove merge comment, clarify wording, and drop useless return value #1026

jrconlin opened this issue Sep 26, 2017 · 0 comments
Assignees

Comments

@jrconlin
Copy link
Member

After #1023, there was some discussion about the second get_uaid call in WebPushRouter.route_notification. This question was clarified, however the comment was determined to vague about what was happening.

In addition, there's a useless bool that was being returned from Message.store_message that could be removed to prevent similar confusion later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants