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

[PE-D] Changing the notebook.txt /pin and /archive leads to bugs when starting program #159

Closed
nus-pe-bot opened this issue Oct 31, 2020 · 1 comment · Fixed by #192
Closed
Assignees
Labels
severity.Low A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situat type.Bug A bug
Milestone

Comments

@nus-pe-bot
Copy link

Steps to reproduce:

  1. go to notebook.txt: and change the /pin and /archive

image.png

  1. save and run the program

  2. Requires me to enter some note at the start now every time i run the program

image.png

After closing the note write:

image.png


Labels: severity.High type.FunctionalityBug
original: AndreWongZH/ped#7

@Chongjx
Copy link
Collaborator

Chongjx commented Oct 31, 2020

@prachi2023 The step to reproduce the bug is wrong, it should be changing the title manually in notebook.txt. Thus, when loading, it attempts to read an non-existent txt file and the content is empty. Maybe can do an error handling. When loading, if it is unable to find the note txt file, skip it. Also do a save all at the exit of Notus.

@Chongjx Chongjx added severity.Low A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situat type.Bug A bug labels Oct 31, 2020
@Chongjx Chongjx added this to the v2.1 milestone Oct 31, 2020
@prachi2023 prachi2023 linked a pull request Nov 4, 2020 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
severity.Low A flaw that is unlikely to affect normal operations of the product. Appears only in very rare situat type.Bug A bug
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants