User Details
- User Since
- Jan 28 2019, 5:32 PM (210 w, 2 d)
Mar 9 2020
Just like that, yeah. Maybe with @JvmOverloads too, but that's a minor thing
Mar 8 2020
This should not be automatically selected. Instead it should default to production and the API base constructor should take a URL to change the base request URL.
Dec 5 2019
- Fix stuff that got asked for
Dec 4 2019
Oct 27 2019
This is why we need a versioned API
Oct 22 2019
How about making "in stack" black (and gray in dark theme ;))?
Oct 21 2019
I'm liking the original Clovis proposal. However:
Oct 4 2019
Sep 7 2019
Superseded by D259
Jul 3 2019
I meant a callback. So you pass a callback into the code to request an object, and it is called with the new object as the only parameter when the cache update is complete, if and only if the object is actually different
Tbh I think we need some system to inform the client when an object has been updated. So that the UI can be updated when an update is performed without closing and reopening something (if you close and reopen a page the old data is displayed because the UI is loaded before the cache is refreshed
Oh, I understand. It's committed to webcl (web client), but its a fix for the webview app, which loads webcl.
Anyway this doesn't actually fix the issue. Other stuff fails and I won't bother.
- Remove extraneous System.out.println
Wait, we have an Android app now? And it uses React?
Jul 2 2019
Because I'm lazy
Try making a test to edit the server draft (.save().setText().save()) and check the coverage. The method to edit server draft is never called.
isLocalOnly is only set to false in the else clause.
So it's set to false (meaning uploaded to server) only if we edit the server draft. And we only edit the server draft if it's set to false! So it's never changed.
Fixed in D225
Fix tests
- Don't require active property in JSON as it's not present for Drafts
- Fix draft deletion
Tests pass but an exception is suppressed. I've fixed that exception and I'm fixing the next one, I'll update the diff later. The exceptions are due to errors in main not in the test
No, this is valid
Damn I can't read
Jul 1 2019
- Remove extraneous character in comment
- Remove test code
Can we get this or something similar in please, I've been trying for months and these deprecation warnings are annoying me
May 18 2019
May 13 2019
WyldBot, why not fix the library then? I was going to do that but y'all told me the server isn't stable yet. So why aren't we versioning the api server endpoints?
May 12 2019
I think you should focus on using wflib rather than rewriting the array.
Apr 27 2019
Wel I have no idea how that works. You can commandeer this and fix it as I don't know what's wrong or how to fix it
Apr 26 2019
So there was never a PGP signature anyway? It only ever existed on your local machine?
Apr 25 2019
Dude. That commit was never signed. I just checked out arc revision 877 which is your original commit. I ran git show HEAD but there's no signature below the commit message.
Remove programming by exception
This comes from D223. I'll deal with it's issues now.
Right, it applies over master now.
Fix minimal-json JAR not being included.
Arc broke. I didn't update the copyrights.
I haven't removed your commits....
Fix minimal-json JAR not being included.
I just searched phabricator the commit is not here, even in unmerged diffs. I'll abandon this and open a new one against master, I suppose
I have no idea when no.
And yes, they must be duplicated.
Fix background invocation
Fix stuff
- Fix bad copyrights
- Don't start API when --norun is specified
- Use rm -f instead of test -f
Apr 24 2019
Lint lines are too long. Check my inline comment in AlertChildPageViewController.xib:2
Oh ok.
Apr 23 2019
Apr 20 2019
Can you check that the unit tests pass? Gradle skipped them so I guess they do but just to be sure
Apr 19 2019
Again my source has been nuked. Can you just merge the diff?
Hmmm I've nuked my source tree so if it's not there it's not there. Is there a way to just apply the diff?
Something is needed, at least. check is the wrong command; we should use either spotbugsMain spotbugsTest or spotbugsMain.
Apr 3 2019
Tested by manually setting localStorage.blockedUsers = "", now it works.