| |
@@ -16,8 +16,6 @@
|
| |
from .forms import MessageSendForm, MessageRecipientForm
|
| |
from .models import Message, BLACKLIST_HMAC_SALT, BlacklistedEmail, strip_email
|
| |
|
| |
- import fedmsg
|
| |
-
|
| |
logger = logging.getLogger(__name__)
|
| |
|
| |
|
| |
@@ -147,6 +145,5 @@
|
| |
if msg.sender_approved_public and msg.recipient_approved_public:
|
| |
sender_name = msg.sender_name if msg.sender_approved_public_named else "Anonymous"
|
| |
recipient_name = msg.recipient_name if msg.recipient_approved_public_named else "Anonymous"
|
| |
- fedmsg.publish(topic='happinesspacket.send', msg={'id': msg.identifier, 'sender_name': sender_name, 'recipient_name': recipient_name, 'text': msg.message,})
|
| |
messages.success(self.request, "Your choices have been saved.")
|
| |
return HttpResponseRedirect(self.request.path)
|
| |
shraddhaag
commented 5 years ago
Q - What is a summary of your change?
A - This commit removes Fedmsg as a dependency for the project.
Q - Why is this change helpful?
A - Since Fedmsg is deprecated in favour of fedora-messaging, this commit removes the Fedmsg dependency for the project.
This solves ticket #66 and #69 and is the first step of many for #96. Since fedmsg is used only for assigning badges, it doesn't conflict with either of these issues and restores them back to full functionality. It's important to get this functionality in the codebase so others working on the project can work on things forward from here.
Note - The test suite was successfully passed.