search

blackmirag3 / pe

0 stars 0 forks source link

Currency converting image not loading in pdf #4

Open blackmirag3 opened 3 months ago

blackmirag3 commented 3 months ago

Screenshot 2024-04-19 at 4.39.40 PM.png I think this may not be intended, as the png files do not show in the pdf

nus-pe-bot commented 3 months ago

Team's Response

No details provided by team.

The 'Original' Bug

[The team marked this bug as a duplicate of the following bug]

Invalid Images in document

Note from the teaching team: This bug was reported during the Part II (Evaluating Documents) stage of the PE. You may reject this bug if it is not related to the quality of documentation.

image.png

The images in Section 3.21 are not displayed. There are only two names: CurrencyConverter_ChangeUSD.PNG, CurrencyConverter_SameCurrency.PNG

[original: nus-cs2113-AY2324S2/pe-interim#1988] [original labels: type.DocumentationBug severity.High]

Their Response to the 'Original' Bug

[This is the team's response to the above 'original' bug]

Hi, thank you for pointing this out and we acknowledge that the images are missing. However, we believe that even without these images, the usage of this change currency command would be well described within the UG with the current instructions, and with the relevant error/success messages for the different possible inputs in the application itself, the users need not rely too heavily on the screenshots for the expected output.

Therefore, we have classified this severity under Medium instead.

Items for the Tester to Verify

:question: Issue duplicate status

Team chose to mark this issue as a duplicate of another issue (as explained in the Team's response above)

Reason for disagreement: [replace this with your explanation]

## :question: Issue severity Team chose [`severity.Medium`] Originally [`severity.Low`] - [ ] I disagree **Reason for disagreement:** [replace this with your explanation]