Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Text expansion shortcut #1057

Merged
merged 7 commits into from
Apr 19, 2017
Merged

Text expansion shortcut #1057

merged 7 commits into from
Apr 19, 2017

Conversation

tangollama
Copy link
Member

Based entirely on the work of @eddiesholl in 2016, this is the completed feature for review.

Changes proposed in this pull request:

  • Added model for text-expansion
  • Route for CRUD functions of text expansion
  • Navigation and locale changes
  • Inclusion of the new component in patients, procedures, visits, labs, imaging, incidents, appointments, and inventory

screen shot 2017-04-18 at 3 10 49 pm

screen shot 2017-04-18 at 3 10 21 pm

cc @HospitalRun/core-maintainers

Copy link
Member

@jkleinsc jkleinsc left a comment

Choose a reason for hiding this comment

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

@tangollama looks good overall, but the translations need to be camelCased instead of using underscores. Can you also merge in the latest from master?

@@ -208,6 +208,17 @@ export default {
messages: { roleSaved: 'The {{roleName}} role has been saved.' },
titles: { roleSaved: 'Role Saved' }
},
text_replacements: {
Copy link
Member

Choose a reason for hiding this comment

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

@tangollama these need to be camel cased.

@tangollama
Copy link
Member Author

@jkleinsc bump. Changes checked in.

@jkleinsc jkleinsc merged commit ed9b002 into master Apr 19, 2017
@jkleinsc jkleinsc deleted the text-expansion-shortcut branch April 19, 2017 14:58
@sthutchinson
Copy link

I like this. It would really help with notes._

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants