Skip to content
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

Mesage Format: Too many spaces above @pings may lead to confusion. #26

Closed
Naheulf opened this issue Jun 17, 2023 · 1 comment
Closed

Comments

@Naheulf
Copy link
Contributor

Naheulf commented Jun 17, 2023

Description of the Issue

There is too many blank lines between the "normal" message part and the list of @Pings

Steps to Reproduce the Issue

  1. Register a feed in Amelia
  2. Set at least one subscribe ping to that feed
    3a. Wait for the feed to have at least two updates in less that 10minutes.
    OR
    3b. Spam the /test command to make amelia send few chapter update message in a row. You must do that from a different discord channel so the message won't interleaves

Expected Behavior

@notifications should be closer to the related message than to the next message

Actual Behavior

image
Look how the first @ping look closer to the second message.

Naheulf added a commit to Naheulf/Amelia that referenced this issue Jun 17, 2023

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@ShindouMihou
Copy link
Member

Fixed the doubly newlines.

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

Successfully merging a pull request may close this issue.

2 participants