Paul Fitzpatrick
Paul Fitzpatrick
@rambip it would be great if you could contribute. The map code is pretty short and self contained, just some html/js/css https://github.com/gristlabs/grist-widget/tree/master/map
Node has been a bit of an outlier in terms of support for proxy variables. Since https://github.com/nodejs/undici/pull/2994 (undici is what node uses for its native fetch implementation) that may be...
There is a method in the code that is intended to delete a user from anything they belong to: https://github.com/gristlabs/grist-core/blob/8853e095bb09afdfaa984de271736b6edd36114c/app/gen-server/lib/Doom.ts#L92 Nobody has built a UI for it yet though.
There is an admin panel these days @almereyda, at /admin. So far it has accumulated diagnostics but no one has put configuration there yet. I expect continued evolution, and would...
> Fixed, thanks @nbush! Neat! Looks like there's a test that needs updating after the text change @fflorent ?
The "Housekeeper" might be a natural place for this code to live: https://github.com/gristlabs/grist-core/blob/ec0b88572333b87102e25bfa6a78e00b101a578c/app/gen-server/lib/Housekeeper.ts#L29-L41 It may be simpler than other housekeeping tasks, since it is scoped to an individual doc worker.
> I have rather considered adding a timer in ActiveDoc Hmm you'll need to work carefully. When a document is unused for some time, its `ActiveDoc` will currently shut down...
The bulk of RAM is held by the sandbox associated with the ActiveDoc so holding ActiveDocs around after sandboxes shut down is certainly not ruled out, just a significant change.
Hi @fflorent, sorry for the delay in reviewing this! Dmitry just hasn't had time, and likely won't for a while. @SleepyLeslie has offered to take it on.
There is a related TODO over here: https://github.com/gristlabs/grist-core/blob/cac05de159a99e6cece1a48b31748d3cfdf39fbe/app/server/lib/ActionHistoryImpl.ts#L712-L717 The strategy proposed here feels a bit aggressive? If a vacuum happens to be in progress, might a document become non-responsive for...