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

Several improvements to get and modify. #115

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

glebius
Copy link

@glebius glebius commented Dec 21, 2023

This boils down to the following changes:

  1. modify: add -Q, combined with -C quotes most recent comments into editor.
  2. get: lots of improvements with printing real names and readable dates
  3. get: combine bug history with bug comments in chronological order, just like Bugzilla web does

@glebius glebius force-pushed the master branch 3 times, most recently from 3eddb2c to 3dd9b03 Compare December 27, 2023 21:04
@glebius glebius changed the title Add a functionality to quote recent comments in your replies within interactive editor. Fixes a bug before that. Several improvements to get and modify. Dec 27, 2023
Use default '%+ UTC' as Bugzilla WebUI does.  Provide key/config option
to override that.
Keep track of real names as we parse the bug and reuse the dictionary
as we print list of comments.  Should work unless somebody has commented
and then unsubscribed from Cc, in that case fallback to just email.
With this change and preceding real name and date/time changes a comment
header gets as close to the WebUI as possible.
Not a useful header and makes it closer to WebUI.
in a chronological order, just like WebUI does.  Try our best with
pretty-printing bug changes and fall back to printing the hash in
the worst case.
Also, make these two extensions automatically raise the patch flag.
Also, make the patch flag set content type to text/plain, unless
mimetypes was able to say something.

P.S. Python's mimetypes module is really helpless.
@arthurzam arthurzam self-requested a review September 27, 2024 19:40
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