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

Housekeeping and general fix-ups before publication #2

Merged
merged 8 commits into from
Jul 21, 2017
Merged

Housekeeping and general fix-ups before publication #2

merged 8 commits into from
Jul 21, 2017

Conversation

aaronbassett
Copy link
Contributor

@aaronbassett aaronbassett commented Jul 1, 2017 •
edited
Loading

  • Fix pep8 errors & remove tabs from Python code
  • Style front end
  • Move all JavaScript into external file
  • Use jQuery best practices / remove JS attributes from HTML
  • Update client side error checking
  • Move files into correct directories and remove clean up old files
  • Add tests & Travis
  • Update Python dependencies and add checking
  • Update README
  • Do E2E testing and QA
  • 👍 Ready to Merge

@aaronbassett aaronbassett mentioned this pull request Jul 1, 2017
Closed
@sammachin sammachin merged commit 433ca16 into master Jul 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
don't merge enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aaronbassett @sammachin