Performed refactoring for the in4189 course. #23

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

jdonkervliet wants to merge 1 commit into delftsre:master from jdonkervliet:master

jdonkervliet commented Jan 17, 2015

From Stefan Hugtenburg and Jesse Donkervliet


          Performed refactoring for the in4189 course.

4daa52d

Author

jdonkervliet commented Jan 17, 2015

We have performed 2 main refactorings:

A huge reduction of cyclic dependencies in the db package.
Pulling the Velocity framework into the ProjectsView class. Hereby removing the hardcoded html that was generated by the Java class.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet