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

Automatically fill out the title of patch based on attached thread #16

Closed
JelteF opened this issue Jan 31, 2025 · 0 comments · Fixed by #30
Closed

Automatically fill out the title of patch based on attached thread #16

JelteF opened this issue Jan 31, 2025 · 0 comments · Fixed by #30
Assignees

Comments

@JelteF
Copy link
Collaborator

JelteF commented Jan 31, 2025

This requires two things:

  1. Move choosing the thread to the top of the patch creation form
  2. After selecting a patch from the dropdown, also prefill the title (stripping common prefixes like Re:/fwd:/[PATCH] etc)
@JelteF JelteF self-assigned this Feb 5, 2025
@JelteF JelteF closed this as completed in f1329ab Feb 15, 2025
JelteF added a commit that referenced this issue Feb 16, 2025
It's a bit silly that you have to manually enter something in the
description, when you just selected a thing with that title from the
thread search modal. So this starts auto-filling the description. This
only happens when the field is empty, and will strip some common
prefixes like `Re:` and `[PATCH]`.

Fixes #16
JelteF added a commit that referenced this issue Feb 16, 2025
It's a bit silly that you have to manually enter something in the
description, when you just selected a thing with that title from the
thread search modal. So this starts auto-filling the description. This
only happens when the field is empty, and will strip some common
prefixes like `Re:` and `[PATCH]`.

Fixes #16
JelteF added a commit that referenced this issue Mar 2, 2025
In fc16044 the patch creation form was based on the edit form. This
worked fine, but the newly added fields were not actually saved in the
database (oops). This fixes that.

Related to #16
JelteF added a commit that referenced this issue Mar 2, 2025
In fc16044 the patch creation form was based on the edit form. This
worked fine, but the newly added fields were not actually saved in the
database (oops). This fixes that.

Related to #16
JelteF added a commit that referenced this issue Mar 4, 2025
It's a bit silly that you have to manually enter something in the
description, when you just selected a thing with that title from the
thread search modal. So this starts auto-filling the description. This
only happens when the field is empty, and will strip some common
prefixes like `Re:` and `[PATCH]`.

Fixes #16
JelteF added a commit that referenced this issue Mar 4, 2025
In fc16044 the patch creation form was based on the edit form. This
worked fine, but the newly added fields were not actually saved in the
database (oops). This fixes that.

Related to #16
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.

1 participant