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

A document may refer to a non a existing document #138

Closed
benel opened this issue Dec 5, 2023 · 5 comments
Closed

A document may refer to a non a existing document #138

benel opened this issue Dec 5, 2023 · 5 comments
Labels

Comments

@benel
Copy link
Member

benel commented Dec 5, 2023

Displaying a document that includes a non existing document results in the following exception being raised:

TypeError: null is not an object (evaluating 'x._id')
@benel benel added the fix label Dec 5, 2023
@benel
Copy link
Member Author

benel commented Dec 6, 2023

Please note that this fix could be the first requirement to a feature that would pull the document from a different server (see #141).

@GwenRiou
Copy link
Contributor

Created page for 404 error

@GwenRiou
Copy link
Contributor

Image

@benel
Copy link
Member Author

benel commented Mar 27, 2024

@GwenRiou Thank you for your mockup.
I think however that the issue your project is interested in is #115 rather than this one.

This issue (138) relates with the case when the focused document does exist (in the center of the interface) and should be displayed but refers to a non-existing document (in the left margin).

@benel
Copy link
Member Author

benel commented Oct 10, 2024

Recent refactoring changed the exception into an invalid displayed page.
This bug has been experienced by a student of @jfalip after deleting a non terminal document.

@benel benel closed this as completed in 93f4280 Oct 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Status: Suggéré
Development

No branches or pull requests

2 participants