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

PR 69 #101

Merged
merged 23 commits into from
Apr 9, 2019
Merged

PR 69 #101

merged 23 commits into from
Apr 9, 2019

Conversation

JohanAanesen
Copy link
Owner

Shit added:

  • Reviews overview (just a table will be changed further after the review pool is done)
  • Typo Repositroy is fixed to repository
  • Count issue for reviews done fixed
  • Fixed radiobuttons not activating issue
  • Merged with Brede's branch for the mail functionality

Johan Aanesen and others added 23 commits March 27, 2019 17:24
Updated notes and go fmt
5ba5043
#Beta-Johan added reviews overview for assignments
10c10e1
Merge branch 'master' of github.com:JohanAanesen/NTNU-Bachelor-Manage…
6c0d33a 
…ment-System-For-CS-Assignments into johan-manual-review-activation

merge
#Beta-Johan go fmt
96eb057
#Beta-Johan fixed typo in package 'repositroy' to repository
3bcb779
#beta-johan update review overview
3398f72
#72 fixing count issue for reviews done
ef224a2
#Beta-Johan moved back button
4bd9ea0
#85 Fixed issue where only radiobuttons on right-most review would re…
2d83239 
…nder as checked
#beta-johan changed title for revierw overview page
93c1913
redundant div row tag
1613136
#beta-johan nani
ffc4ecf
beta-brede - update - updated system with the new name
bc9a3bd
Merge branch 'master' of github.com:JohanAanesen/CSAMS into beta-brede
bc02933
beta-brede - update - notes
cbb92ac
beta-brede - update - added possible solution for #97
9c5663a
beta-brede - update - changed forgotten_password to validation #97
9b25852
beta-brede - refactor - changed name on forgotten_password to validation
a0595a6
beta-brede - new - added files for user_pending
aa13e41
beta-brede - update - added service and repository + model for users_…
86f38d5 
…pending #97
beta-brede - update - new users have to confirm wmail before being cr…
cccadf6 
…eated #97
merge
afde13c
Mailservice merge, repositroy typo fix and other small features
00d87f5
BredeFK
BredeFK approved these changes Apr 9, 2019
View reviewed changes
Copy link
Collaborator

@BredeFK BredeFK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@BredeFK BredeFK merged commit 913b182 into master Apr 9, 2019
@JohanAanesen JohanAanesen deleted the johan-manual-review-activation branch April 10, 2019 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BredeFK BredeFK BredeFK approved these changes

@kongebra kongebra kongebra approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@JohanAanesen @kongebra @BredeFK