-
Notifications
You must be signed in to change notification settings - Fork 8
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
Ui live indicator #94
Conversation
The admin ui now has toggleable live symbles within the $scope.menu array, however i cant find a way of getting that information from the cg controllers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Versioning in package.json needs to be discussed before merging.
package.json
Outdated
@@ -1,13 +1,15 @@ | |||
{ | |||
"name": "la1tv-roses-graphics", | |||
"version": "1.0.0", | |||
"version": "2018.0.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When was the version change from SemVer ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we werent semvering tbf, plus i feel it makes more sense to version on the year and minor/patching
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets not change the versioning scheme in this PR regarding UI live indicators we can switch to this at a later date
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fair play, think its worth changing though
public/js/dashboard.js
Outdated
@@ -263,6 +289,22 @@ app.controller('lowerThirdsCGController', ['$scope', 'localStorageService', 'soc | |||
|
|||
var stored = localStorageService.get('lower_thirds'); | |||
|
|||
const showLiveLT = $scope => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Expanding showLiveLT
to showLiveLowerThird
etc helps with readability of code.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
argeed will fix in a mo
added netball aswell now its been pulled in |
changed back to semver now |
No description provided.