Skip to content

Commit

Permalink
Updated W18 final remarks
Browse files Browse the repository at this point in the history
  • Loading branch information
nathan-inge authored Mar 16, 2018
1 parent 888aaaf commit 8a6d5d4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -87,4 +87,4 @@ To the next group of students, we would reccomend reading as much of the code as

W18 Final Remarks:

To the next group of students, this quarter we primarily spent our time converting the parsing structure from substrings to using a parsing library called JSOUP. We recommend looking at `loadCoursesJsoup` located in `UCSBCurriculumSearch.java`. This function (and its helpers) utilizes the JSOUP library and is where the parsing of the HTML happens. We also recommend looking at the newly refactored tests in `UCSBCurriculumSearchTest.java` to see the functionality of the parsing. The GUI interface is still broken and we recommend not investing too much time with it, as the future of the project leans more towards the API interface ( the project *cs56-webapps-curriculum* may eventually end up combining with this one).
To the next group of students, this quarter we primarily spent our time converting the parsing structure from substrings to using a parsing library called JSOUP. We recommend looking at `loadCoursesJsoup` located in `UCSBCurriculumSearch.java`. This function (and its helpers) utilizes the JSOUP library and is where the parsing of the HTML happens. We also recommend looking at the newly refactored tests in `UCSBCurriculumSearchTest.java` to see the functionality of the parsing. The GUI interface is still broken and we recommend not investing too much time with it, as the future of the project leans more towards the API interface ( the project *cs56-webapps-curriculum* may eventually end up combining with this one). A good issue to start on would be refactoring the `getPage` method in `UCSBCurriculumSearch.java` and introducing a library to get the HTML from the web page in a more reliable way. The current `gePage` sometimes just doesn't return the page. Try using the Geb web automation library to get the contents of the search page instead of hard coding everything like in `getPage`'s current implementation.

0 comments on commit 8a6d5d4

Please sign in to comment.