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

add history to Console (plus some minor tweaks) #256

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

rjbs
Copy link
Member

@rjbs rjbs commented Jan 31, 2025

No description provided.

rjbs added 8 commits January 30, 2025 21:35
This lets us go back and look at more things about the messages, which
will be useful for investigating "alts" that are not normally displayed
in the console.
This will let you see things in history, including with "alts".
Since we can default to "this channel", make it optional.  Format is
more likely to be useful, so put channel name at the end.
In 7307ef8 and its parent, Page changed to split the message on \s+ and
then join later with \N{SPACE}.  This turned vertical whitespace into
horizontal whitespace.  This broke a test, but we didn't notice. 🙁

I'd rather preserve space, but that's a bit more work and I'm not very
motivated.  So, this just fixes the test and keeps blank lines working.
I feel like I have done this before...
@rjbs rjbs requested a review from lerlacher-fm January 31, 2025 05:50
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 this pull request may close these issues.

1 participant