Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Loading time / abstraction? #35

Open
M123-dev opened this issue Sep 20, 2021 · 4 comments
Open

Loading time / abstraction? #35

M123-dev opened this issue Sep 20, 2021 · 4 comments

Comments

@M123-dev
Copy link
Member

For me the documentation has horrible loading times, the index.html file has 554.644 lines of computer generated code which will increase in the future with documenatation from robotoff and other features.
(The 3300 lines in off-pm-collection.js should be okay for now I think).

I would suggest that we divide the documentation a bit more. But that would probably mean that we have to move away from docgen

Any opinions on this

@teolemon
Copy link
Member

Is that on the documentation hosted on GitHub pages ?

@M123-dev
Copy link
Member Author

I took a closer look and it seems that this only occurs with Firefox, interesting. (Mobile and PC)
It gets stuck at the "TLS-handshake with static.openfoodfacts.org", and currently even longer than usual.

With Chrome, however, everything runs smoothly.

500 thousand lines are still a lot though 😄

@ocervell
Copy link
Contributor

postmanerator seems more friendly than docgen, I can give it a try when I get some time to do it :)

@ocervell
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

No branches or pull requests

3 participants