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

GET /api/texts/ route should return only what the texts page needs. #98

Open
mrchrmn opened this issue Dec 30, 2021 · 0 comments
Open
Labels
enhancement New feature or request question Further information is requested

Comments

@mrchrmn
Copy link
Member

mrchrmn commented Dec 30, 2021

At this moment, all information about a text, including the full text, is transferred from the server for the text list page.

This may not be problematic now, but it could become a problem when the list grows and the texts become longer (think book chapters), particularly on mobile.

I suggest that for the list the query returns only the id, title, and truncated text. When clicked, the full information, including full text, is loaded and stored in the currenttextState.

@mrchrmn mrchrmn added enhancement New feature or request question Further information is requested labels Dec 30, 2021
@mrchrmn mrchrmn changed the title GET /api/texts/ route should return only what the tests page needs. GET /api/texts/ route should return only what the texts page needs. Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

1 participant