This is 100 years old and I'm no longer sure what it's describing or how to reproduce it. Feel free to file a new issue if this is still a problem. Note that Audit is churning a lot right now (see T10978) so the behavior may change in the near term. This may also be a "viewer audit status" vs "commit overall audit status" issue in some sense (T9482 / T6024).
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Jan 11 2017
We no longer offer this option at HEAD. I'm moving us away from the self raise + self accept workflow to align better with Differential and because the state is unintuitive.
I believe D17183 fixes this. If you're still seeing issues after upgrading past that change, file a new issue with modern reproduction instructions.
D17183 fixes half of this: previews are now sorted in creation order, from oldest (top) to newest (bottom). This is the same order used in Differential. We use this order for previews because it makes it easier to review recently added comments while working through a review/audit.
I'm just going to presume this is resolved, in some sense, by D17181, because you can now remove auditors regardless of other outcomes.
Although I earlier (in T6630#85282) felt like it was reasonable to consider separating Audit and Diffusion, there are now more existing or planned integration points and I think this approach is significantly less reasonable than the "merge them" approach. It also doesn't seem to be causing any problems on real installs.
Jan 10 2017
I believe this was resolved by the introduction of the "query overheating" mechanism, described in T11773.
I'm going to merge this into T5815, which is a more general discussion of action queues.
I can't reproduce this; we can't move forward without reproduction steps.
I'm just going to merge this into T10448, this task doesn't have anything not already covered there.
- DiffusionCommitQuery has had the "good" version for a while.
- We're nuking Releeph at some point.
Dec 29 2016
Dec 12 2016
"Blame Revision" was a freeform text field at Facebook (from before DiffCamp, even) where you were ideally supposed to enter the revision or commit that introduced the bug you're fixing, if you were fixing a bug introduced by some known change.
Dec 9 2016
Dec 8 2016
Nov 15 2016
Nov 3 2016
Nov 1 2016
Also interested in this. Our team plans on using the audit tool for code reviews, it be good if when a developer has made any suggested changes and re-committed that the old open audit could be closed/linked to the latest commit rather than having to comment on the old one with a link to the changed copy and just closing the old one manually.
Oct 17 2016
Oct 8 2016
How about this, it's solved or not?
Sep 22 2016
Sep 15 2016
Sep 8 2016
We're looking for a similar concept for Reviewers:
Even if the Author of a review is one of the Owners, we still want to automatically add that package.
This is mostly just T1125 - we want to make it easier to assign reviewers, and Owners has good UI for selecting paths.
Aug 7 2016
FWIW this exact set of reproduction steps no longer works. However, an Owners package set to generate an audit, but without any members, will be unclosable as well.