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.

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

Refactoring & bug-fixes #19

Merged
merged 342 commits into from
May 23, 2017
Merged

Refactoring & bug-fixes #19

merged 342 commits into from
May 23, 2017

Conversation

nicho90
Copy link
Contributor

@nicho90 nicho90 commented May 23, 2017

No description provided.

nicho90 and others added 30 commits April 6, 2017 14:53
(will also be done for user-client)
Added missing Concerns to Review Page
Improved Layout
added config.js support
renamed Reviews to Reviewers
regarding renamed reviewers
added „general_comment“ and „reviewer(s)“ to submission-view
Added attribute „updated“ to Revisions to show the submitted timestamp
inside the user-/admin-clients
- integrated navController inside mainController
- fixed promises in documentDetailsController
- fixed editDocument-template
- updated apiControllers
- updated routes
- updated queries
refactored all text on site into languages.js
Fixed infinite loading screen after User Creation.
SpeckiJ and others added 26 commits May 7, 2017 15:11
rephrased to proper English.
Conflicts:
	public/member-client/js/modules/languages.js
- Moved the "Start Review"/"Open Review" Buttons to top of the page
- Added subtitles to Files at download (provided by fhillen)
- deleted config.js and replaced it with an .env-file, following the
dotenv-guidline
- Code-cleaning
- bug-fixes in email-templates and notifications
- Added path variables for the member- & user-client to the .env file
- Fixed email-templates
- Updated README
Updated languages.js and related controllers in user- and member-client
@nicho90 nicho90 added the bug label May 23, 2017
@nicho90 nicho90 self-assigned this May 23, 2017
@nicho90 nicho90 merged commit b83738c into master May 23, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants