- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
All Stories
Aug 18 2023
Looks good to me.
Ouch ! Thanks for reporting ! I'll have a look on UI examples (these pages are really cool !) to improve the proposal !
BTW, I'll be away next week so... This will not move forward right now...
Great, let's prepare for landing ! BTW, I like your images, espacially this one...
Yeah anyway this looks great
In D25409#11652, @avivey wrote:I think phutil_nonempty_string fails faster then strlen (strlen allowing automatic conversion of string-like things to string and phutil_nonempty_string doesn't), but I'm a little confused right now...
Switch to using phutil_nonempty_string
SUPER NICE \o/ THANKS
Tested with and without D25387.
I think phutil_nonempty_string fails faster then strlen (strlen allowing automatic conversion of string-like things to string and phutil_nonempty_string doesn't), but I'm a little confused right now...
In D25409#11648, @avivey wrote:phutil_nonempty_string is basically the exact same thing, but with stricter type control
phutil_nonempty_string is basically the exact same thing, but with stricter type control
This happened some minutes ago to our friend @waldyrious and I was able to reproduce it on my local computer. I've attached my reproducing instructions.
Maybe to be it consistent with "super-rebranded-product/" :D I don't know
I don't understand why the pht was there - it's a "path", why would it be translated?
Yeah fixed in D25104
I've just:
Oops, sorry, I was not on master. I was actually testing out D25079. This is already fixed on master, by @valerio.bozzolan.
git rebase master
Let's visually mark this as needing still some work
Thanks bob! You are a good minion. As discussed previously, this is used only in one point and without any argument:
Great news ! I'll land it right now !
I've tested this in place with my eyes on a friend's laptop with PHP 8.1 in the very same problem
Nice job, my little and cute bob.. OH NO! a super-important regression! all of you minions will be fired soon!1!
OK I've found that this is about the legacy Conduit entry point for Paste.
I'm not able to trigger that part of the code.
Tested with and without a value, thaaanks \o/