search

shamesjen / pe

0 stars 0 forks source link

DG and UG are not PDF-friendly #4

Open shamesjen opened 6 months ago

shamesjen commented 6 months ago

image.png

Description

As shown in the image above, when I copied the sample command to add a seller, due to the way the command was wrapped, it was missing a space.

Steps to reproduce

  1. Launch the application
  2. Run addSeller n/John Doe p/98765432 e/johnd@example.com type/Hdb street/Clementi Ave 2 blk/311 level/02 unitNo/25 postal/578578 price/999999999.

    Expected behaviour

    At step 2, John Doe should be added as a seller

    Actual behaviour

    At step 2, there is an error regarding the command format.

    Reason for severity

    A user may never be able add a seller, however, if the user types out everything himself, or checks through the command, the user may be able to find the issue with the copied command.

    Ideas

    The wrapping of the text does not include a space between 25 and postal.

nus-se-script commented 6 months ago

Team's Response

This seems to be an issue with your PDF reader. We have tested it with Google Chrome's in-built PDF reader and it works perfectly fine. image.png

Items for the Tester to Verify

:question: Issue response

Team chose [response.Rejected]

Reason for disagreement: Good point by the dev team. I procured a windows device and tested the Google Chrome in-built PDF reader that the dev team suggested
video:https://raw.githubusercontent.com/shamesjen/pe/main/files/45e468f5-e99e-453c-aa53-4d4eac7ebb7b.mp4

However, I realised that even when using the suggested devices the dev team recommended, I was unable to achieve the correct result. I believe the dev team rejected this bug without testing or trying to replicate the bug.