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

Musician list resets after addb #4

Open
wjayee opened this issue Nov 17, 2023 · 1 comment
Open

Musician list resets after addb #4

wjayee opened this issue Nov 17, 2023 · 1 comment

Comments

@wjayee
Copy link
Owner

wjayee commented Nov 17, 2023

If i run addb n/test2 after running find n/john, the list resets even though adding bands it does not affect musicians

reset list bug.png

@nus-pe-bot
Copy link

nus-pe-bot commented Nov 20, 2023

Team's Response

Thank you for pointing this out. Currently, this is the expected behavior of our app. We believe that the UI could have been more intuitive for the user and we plan to improve on this in the future. In fact, this falls under the last point of our "Appendix 4: Planned Enhancements" in DG.

Items for the Tester to Verify

❓ Issue response

Team chose [response.Rejected]

  • I disagree

Reason for disagreement:
image.png

Based on the last point in the planned enhancement section, it states that the UI is not intuitive and the example given was that the band panel does not change when filtering the musicians. From what I understand, its about that the band panel and musician's panel are not affected by each other's commands, like how the band panel is not affected by the filtering of musicians.

However, my concern was that the band panel should not affect the musician's panel, which it did, when I added a band to the list of bands. Basically, the musician's panel was not edited or affected in any way but it's filter was removed after I added a band. Additionally, it is not stated in the UG that the addition of bands would reset any filter on the musician's list.

Thus, I believe that this does not fall under the last point in the planned enhancement section in the DG.


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

No branches or pull requests

2 participants