Paul Fitzpatrick

Results 402 comments of Paul Fitzpatrick

> > (apologies, I'm going to use this PR as a guinea pig to examine exactly why our machinery for deploying previews of PRs doesn't function for PRs from forks)...

I'm seeing failures in tests after UsersManager in a derived version of Grist (the one for our SaaS). Haven't looked into why yet, and may be specific to us. ![Screenshot...

Hmm none of this rings a bell @raph-topo. Do you happen to know if these problems existed for you prior to 1.4.0 or is that the first version you tried?

``` at async DocSnapshotInventory.uploadAndAdd (/grist/_build/app/server/lib/DocSnapshots.js:167:9) at async HostedStorageManager._pushToS3 (/grist/_build/app/server/lib/HostedStorageManager.js:717:13) docId=null ``` The `docId=null` is a little surprising to me there, cc'ing @Spoffy in case it rings any bells.

Just tried running Grist locally with minio+redis and things feel normal? Tried replicating B but things worked for me. The description of B sounds almost like an auth problem. Also...

Something odd about this. @fflorent @georgegevoian @Spoffy are any of these symptoms familiar? The only thing that catches my eye is `minio://btp-gt//_docId_.grist`, I presume the `_docId_` part is redaction, but...

@fflorent thanks for looking into this. I haven't had a chance to look closely myself, but the first thing I'd be checking is if a prefix set explicitly to `""`...

Yes, this would be a nice feature to have!

It is a good idea @yohanboniface, and it shouldn't be that much work, the server doesn't care what source the client is sending the material from.

Agree this is important. We have this in our older internal jenkins/phab/arc setup we use for our SaaS, but that is too late for migrations originating here.