馃憤馃憤
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Advanced Search
Mon, Jul 15
Thu, Jul 11
I think we're all aligned here...
Wed, Jul 10
The best way is probably to make an Extension:
Tue, Jul 9
Mon, Jul 8
these pages should have Macro disabled, probably. A user on the install can add more macros that would break the page in some other ways. Probably.
Wed, Jul 3
Once we remove this, we'll need to handle this edge-case:
- I've closed a ticket as "duplicate of T1", but I was wrong and want to mark it as "duplicate of T2" instead (and say it's not a dup of T1).
Tue, Jul 2
Mon, Jul 1
In D25709#19636, @valerio.bozzolan wrote:OK.
@avivey what do you think about a new silly commodity method PhabricatorTransactions::findOneByType(array, string) ?
(BTW, in D25698 we started deprecating remarkup.diviner for RemarkupReferenceController)
Sun, Jun 30
what will this do?
Fri, Jun 28
I'm assuming you're talking about dropping a file in the text area when editing the page, right? Not just dropping it when viewing a wiki page?
Presumably resolved by D25134.
Thu, Jun 27
maybe just add if ($value === null) { continue; } somewhere before the this?
grep for PHUIDiffTableOfContentsItemView in the codebase to see where we might be using this class.
If we can't find the problem, that probably means we have some dead code here that should be removed.
Is there something that's not working here?
:shrug:
(re-opening, there's more work left here)
Jun 26 2024
Do we still need the auto-complete feature? In modern times, the OS has the "insert emoji" feature built-in (at least, Windows and Android and KDE do).
Jun 24 2024
Ok, this looks much better then what we have today, so yay.
Looks good! I want to play with it locally before accepting.
Jun 23 2024
It also collected lots of commits, since May 2023.
In D25697#19176, @20after4 wrote:I obviously failed to run ./bin/celerity map one more time before running arc diff.
There should really be some way to detect and prevent that mistake. It's one I've made probably 100 times before.
Jun 22 2024
Jun 21 2024
Jun 20 2024
What's the current behavior for fallback from cs_CZ? Would it make sense to set the default to en_US in the base class?
Jun 19 2024
What if $cdn has some evil characters? Do we have some escaping, or can it cause anything bad, or do we not care? Looks like we're including a bunch of other config as-is here, so maybe it's ok.
Looks good to me.
Jun 16 2024
I still don't like the solution of using place-holder text for that; But this implementation also adds another DB access which I think is redundant.
Instead of this diff, can you add a modern endpoint that uses the EditEngine/transactions? e.g. maniphest.edit, badge.edit, etc.
That will need a different name (phriction.document.edit?), but will allow full editing capability.
Jun 9 2024
So PHP, very wow
Jun 7 2024
As T15224 suggest, this menu should have the same content as the side-menu on non-mobile view; And in most places, the framework handles that, by actually moving that menu.
So the "right" fix is to find what creates the current side-menu and make sure we call that, or figuring out why they're different here and normally the same.
Jun 6 2024
There may be some room for extra clarity for users in complex organization setups, but this is a rare enough operation (outside of dev) that it's probably not worth worrying about.
Jun 4 2024
Please add some negative tests - things that could trigger this thing, but shouldn't.
Jun 3 2024
Jun 2 2024
I think the EditEngine is what's used to create the actions form, and it has some sense of the object's status (see for example the available actions on Revisions - these change based on the revision's state).
Maybe it can get an additional "field" for this warning, and display it based on task status.
May 29 2024
Very cool!
Don't we have something like phutil_is_integer method somewhere?
May 26 2024
May 25 2024
This is spelling, not formatting.
Using a password in 2024 is a bad advice in general.
May 21 2024
need to also test the result of renderPhabricatorLogo() when no logo set, when a custom logo is set, and when a custom logo is set but hidden (maybe. Maybe the logo file should be Attached to something public, so this situation can't really happen?)
That "don't sign a CLA" article should be titled "don't sign away your copyrights", to be more accurate and less click-bait-y. A CLA may ask you to sign away copyrights (which I feel makes the project non-open-source) ; It doesn't have to.
May 19 2024
May 14 2024
(It's about spam, but it doesn't have anything explicitly sensitive. I've made it public now)
May 12 2024
On a more meta level, Maniphest isn't well-suited to be an entry-form to be filled by a non-expert user; Nuance is/was intended for this use-case.
on the technical level, Herald can't block object creation - it runs after the fact, by the Daemons.
I think the 404 is possibly better then "silently fail without error".
How can we end up with null here?
May 11 2024
Rebase
May 9 2024
May 8 2024
In T15096#12185, @valerio.bozzolan wrote:Another problem dramatically frequent for newcomers (at least in my office).
It's relatively too much easy to start with a similar commit message:
...
(run the full tests, and then count it as my + O1 Accept).
I think the costs of the extra translations are low, but the cost of "the tests are not being translated for developers" are even lower.