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

Refactor code #38

Merged
merged 176 commits into from
Mar 15, 2017
Merged

Refactor code #38

merged 176 commits into from
Mar 15, 2017

Conversation

adeolabadmus
Copy link

Fixes #[replace brackets with the issue number that your pull request addresses].

Changes proposed in this pull request:

  • [list out summary of changes here]
  • [list out summary of changes here]
  • [list out summary of changes here]
  • [etc]

Note: pull requests without proper descriptions may simply be closed without further discussion. We appreciate your contributions, but need to know what you are offering in clearly described format. Thanks! (you can delete this text)

cc @HospitalRun/core-maintainers

greenkeeper bot and others added 30 commits December 11, 2016 12:35
…-cli-stylelint-0.9.0

Update ember-cli-stylelint to the latest version 🚀
…lint-scss-1.4.0

Update stylelint-scss to the latest version 🚀
…ations.js. As of now, the files are up to date with latest commit on actual repo: e33ff61
adeolabadmus and others added 24 commits March 7, 2017 09:28
Fixes from retrospective feedback
fix: fix  new user title bug after a new user has been added
Added deploy branches to .travis.yml
…un#971)

* changed button style

* travis error resolved

* button colour fixed for hover state
* Updating documentation about linters.

* Updating documentation about linters; adding reference to Dockyard's Ember.js style guide.

* Updating documentation about linters; moved links to Style Guides to top.
* create new route print-header

* update navigation bar with print header nav

* adding translation for print header options. complete print header configuration screen

* add translation for print header hav

* add unit test for print header screen
* Initial commit from OasisHospital/frontend

From
https://github.com/OasisHospital/frontend/tree/incident-reporting-1.0

* Fixed missing translations for incident reporting.

* Fixed localization.

* Prod build for testing.

* Added date filter for sortable columns

* Fixed taking pictures with webcam

Fixes issues with using deprecated/removed browser functionality

* Move incident feedback to incident notes

* Changed to use consistent logic to generate friendlyid

Patient, inventory and incident now use the same code to generate
friendlyid

* Added logic to override page limit

* Move common filtering/sorting functionality to mixin

* Removed unneeded code

* Changed completed listing to work like index listing.

* Fixed delete messages

* Updates to incident per latest requirements

Updated incident per requirements here:
https://github.com/HospitalRun/planning/blob/master/Requirements/v1_0/In
cident-Management.md

* Fixed error with Ember inspector loading on load db screen

* Webrtc-adapter was pointing to wrong file

* Added attachments to incidents

Changed code used for patient photos to be reusable so that it can be
used for incident attachments

* Change where incident attachments are saved.

* Fix server uploading of images/attachments

* Added custom forms to incident reporting.

* Fix missed localization.

* Cleaned up incident category

* New lookup list service

We should be pulling looking lists from a service instead of from
parent routes.

* Cleaned up code.

* Fixes for tests that weren't passing.

* Starting incident-reporting acceptance test

* Updated incident acceptance tests

* Refactored operative test to use its own fixture.

* Fixed issues with incident testing.

* Update test for changes to abstract-edit-controller.

* Moving operative test to its own fixture.
Copy link

@Chima1707 Chima1707 left a comment

Choose a reason for hiding this comment

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

Looks good

@Chima1707 Chima1707 merged commit d9e1baf into master Mar 15, 2017
@adeolabadmus adeolabadmus deleted the refactor-code branch March 16, 2017 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet