search

hermanthong / pe

0 stars 0 forks source link

API limit message when too many stocks in view function #9

Open hermanthong opened 3 years ago

hermanthong commented 3 years ago

If the user has many stocks, using view will cause the later stocks to be blocked by the API limit. Should the limit be relaxed for view and bookmarks?

viewapilimit.PNG

nus-pe-bot commented 3 years ago

Team's Response

Most of our functions calls the API, and because of the API request limit, many of our functions will not work if there are too many calls to the API, we agree that this has made it difficult for users to use the app efficiently and has caused some of our functions to constantly display the API limit error especially when the function requires more than 5 calls made to the API to be ran. We have tried to include caveats (such as setting the bookmarks limit to 5) it will still require users to wait for awhile before being able to run any other command. We could only fix these issues if we pay for the premium price for fetching stock data from that API, which we have also stated in our user guide.

Items for the Tester to Verify

:question: Issue severity

Team chose [severity.Medium] Originally [severity.High]

Reason for disagreement: As a user is very likely to have more than 5 stocks in their portfolio, this is a flaw that affects most users and causes major problems for users. i.e., makes the product almost unusable for most users, and it deserves a severity level of high.