Skip to content
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

Diffvy #174

Open
Tarrasch opened this issue May 25, 2012 · 3 comments
Open

Diffvy #174

Tarrasch opened this issue May 25, 2012 · 3 comments
Labels
Low priority View

Comments

@Tarrasch
Copy link
Contributor

Man vill som handledare kunna diffa två vyer, jag kikar lite på möjligheterna för detta, vore jättekul om vi kunde visa detta i presentationen på måndag
@ghost ghost assigned Tarrasch May 25, 2012
Tarrasch added a commit that referenced this issue May 25, 2012
@Tarrasch
Disable "offsets" method in diff-views [#174]
38aa5cb
Tarrasch added a commit that referenced this issue May 25, 2012
@Tarrasch
No support for comments in diffs yet [#174]
a827205
Tarrasch added a commit that referenced this issue May 25, 2012
@Tarrasch
html_safe on methods in diff_helper [#174]
5cb6961
Tarrasch added a commit that referenced this issue May 25, 2012
@Tarrasch
Comment out what breaks in diff_helper [#174]
84d86bd
Tarrasch added a commit that referenced this issue May 25, 2012
@Tarrasch
Routes, controller and view for diffview [#174]
0ba8cd4
@Tarrasch
Copy link
Contributor Author

Ok, satt hela eftermiddagen till och med midnatt, men denna är nästan klar nu! Se gärna screenshots.

Var riktigt meckigt med det dära diff-view biblioteket som dem själva skrivit. Men nu är det nästan klart, har dock inga fina urls som pekar på diffsidorna i själva webbinterfacet ännu.

Aja, nu har jag gjort en vygrej också :)

@Tarrasch
Copy link
Contributor Author

Förresten, routsen är på formen

http://localhost:3000/student/courses/4/lab_groups/3/labs/1/submissions/6/diffs/5

Jag tänkte att ingen bryr sig om hasharna man diffar emot utan snarare bara frågar sig ifall inlämningarna skiljer sig.

@oleander
Copy link
Member

Bra gjort!

@Tarrasch Tarrasch removed their assignment Nov 26, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Low priority View
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@oleander @Tarrasch