Alya Abbott
Alya Abbott
Thanks for the fix! Could you please update your commit message to match the [commit style guidelines](https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages) for the prefix, and avoid typos? You can post a comment when ready...
@alexmv Are you the right person to review this one?
Please add a screenshot for the error state, as in the description for #25764.
Why is "edit history" a link? I don't think it needs to be, and it also doesn't work in my manual testing.
The "Manage uploaded files" link feels out of place to me. Let's try moving it below the list of files, and see if that helps. We can also put it...
Thanks! I reopened the PR and posted a question [on CZO](https://chat.zulip.org/#narrow/stream/9-issues/topic/delete.20attachments.20from.20mesage.20history) for broader feedback.
Is there a reason you've been closing and reopening your PRs?
Yeah, that sounds like a more complicated workflow than you need. In any case, you can always use `git rebase -i` to squash any extra commits.
https://zulip.readthedocs.io/en/latest/git/index.html might help, or asking in #git-help on CZO.
Thanks, the screenshots look good to me! @timabbott do you wan to take a look next?