3 ms·

I hope this doesn't come across badly but: NimForum is an open source project, as such I would encourage you or anyone to help improve its efficiency. I think t

by dom96 5y ago

I hope this doesn't come across badly but: NimForum is an open source project, as such I would encourage you or anyone to help improve its efficiency. I think there is a lot of low hanging fruit here and I'd happily help with any patches that fix them. Back in 2018 when I implemented the basis for this modern reimplementation of the Nim forum I focused on the functionality and a solid CI so that we can easily accept others' contributions without fear of regressions. Unfortunately I had only 2 solid weeks to make this a reality and optimising the code size didn't make the cut.

sim_card_map 5y ago
Improving its efficiency would mean re-writing it entirely, since it's an SPA app with 4 MB of JavaScript.
- dom96 5y ago
  It could, but it doesn't. As many have remarked already, it appears there is a bug that's causing the site to reload Google's recaptcha multiple times. This is a big low hanging fruit fix, another is simply minifying the Forum's JS, there is also bound to be a lot that can be done on the Nim side to reduce the code size generated.
  - cobby 5y ago
    As far as I can see you already have a minification task [1] in your nimble file. May I ask you why you are not using it to save bandwidth both for your sever and your visitors? [1] https://github.com/nim-lang/nimforum/blob/master/nimforum.nimble#L44 https://github.com/nim-lang/nimforum/blob/master/nimforum.ni...
  - cobby 5y ago
    > it appears there is a bug that's causing the site to reload Google's recaptcha multiple times I forgot to mention that it is not a bug. See https://github.com/google/recaptcha/issues/335 https://github.com/google/recaptcha/issues/335 for example (I believe there are others issues like this in the repo)