search

jloh02 / pe

0 stars 0 forks source link

Export/import data is not clear to users, leading to incorrect behavior #11

Open jloh02 opened 1 week ago

jloh02 commented 1 week ago

Although it is stated that only persons will be exported, it is unclear whether events assigned to a person belong to a person or simply exist as an event (In fact, according to code and the json file, would lead advanced users to think they belong to a person). As such this behavior of the assigning is unexpected.

As a user, I would expect that if I were to export persons as a backup, and reimport it back (to restore in a case where someone deletes all my persons, my event links should still be maintained. As the UI is not clear in displaying this, this is a functionality bug as well.

image.png

Steps to reproduce:

  1. Load in default data
  2. export
  3. mass_delete 1 2 3 4 5 6
  4. import ExportedContacts.csv
  5. unassign_event p/2 ev/1 fails

image.png

image.png

nus-pe-script commented 1 week ago

Team's Response

As per the user guide, the import command "Reads the specified file to import from and adds the persons to ClubConnect" and the export command "Exports all persons in ClubConnect into a csv file". It is stated that the functionality of the import and export commands is only meant to import and export the persons respectively, and this functionality does not extend to events. Additionally, as per the developer guide, the import and export commands were designed to facilitate the sharing of contacts, not events, and were not meant to be used as a way to back up data. Thus, the import and export commands do satisfy their intended functionality.

Items for the Tester to Verify

:question: Issue response

Team chose [response.NotInScope]

Reason for disagreement: [replace this with your explanation]