Paul Fitzpatrick

Results 402 comments of Paul Fitzpatrick

I'm sorry review of this PR has stalled. I'm not sure what to do with it. It is an improvement, but accepting and merging it as is would presumably lead...

Yes, I guess documentation may be the best we can do, in practice. One thing we will be working on is a "bootstrap page" for Grist administrators, to inspect the...

Hi @rouja, I've started work on a bootstrap page (now just called boot page) that includes a check of the user running Grist (see https://github.com/gristlabs/grist-core/pull/850) ![Screenshot from 2024-02-15 17-49-52](https://github.com/gristlabs/grist-core/assets/118367/d1180a2c-d345-4418-aca9-39b0a884c1f8) Should...

> I think it's normal that **Grist reachable** is KO because I use grist.127.0.0.1.nip.io and this cannot works from pod. Thanks for testing @rouja !

> @paulfitz @rouja Looks like the need of this PR is now covered by #850 I think it would also be useful to give self-hosters guidance on how to run...

My guess would be that removing the checksum entirely would be a bigger change, and that it may still have utility. Converting the error to a warning seems less disruptive....

Sorry for missing the pings on this thread. I think it would be worth running the tests in test/server/lib/HostedStorageManager.ts with Florent's flag set. There may be an early test in...

Thanks @fflorent for pushing this forward. I'm working through the steps mentioned in https://github.com/gristlabs/grist-core/issues/751#issuecomment-1889557946 * [x] support GRIST_SKIP_REDIS_CHECKSUM_MISMATCH flag * [x] check full battery of Grist Lab's SaaS tests with...

Thanks for reminder about this, working now on getting that last checkbox ticked off.

@fflorent testing in https://github.com/gristlabs/grist-core/issues/751#issuecomment-1998095415 looks fine. It would be ok by Grist Labs for the default behavior to change.