ucsb-cs56-projects / cs56-math-conversion-quiz

-
2 stars 9 forks source link

Fixed Issues 13, 32, 38, and 40 #42

Closed aryamandas closed 7 years ago

aryamandas commented 7 years ago

Pull Request for resolutions to issues #13 , #32 , #38 , and #40 For Issue #38 (Javadoc Comments) comments were added primarily in the following files: Question.java and QuizGui.java Aside from labeling the beginning of the class files with our names and current quarter, the Question.java required javadoc comments on multiple methods including public int getRandomNum(), public String generatePrompt(), public String getRadixString(), and public boolean checkAnswer(). The majority of comments were added to QuizGui.java due to the multiple listener classes for the buttons available in the GUI version of the math quiz. Overall, the addition of javadoc comments throughout the program files took a substantial amount of time. For this reason, we would greatly appreciate it if this particular issue could be boosted to a worth of 100 points! Thanks!

aryamandas commented 7 years ago

UPDATE 11/9/16: Revisited and confirmed issues from previous comment. new Issues resolved: #26, #35, #36, #37, and #40 For Issue #36, we believe all the steps have been taken correctly to resolve this issue, but we were unable to try opening the javadoc in our browser (however, the javadoc files successfully exist on a gh-pages branch in our forked repo). If this has been done correctly, we currently have 575 pts completed, if not 475 pts. completed. If the gh-pages workflow needs to be fixed, we will address this issue tomorrow during lab. Thanks!