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

Feature/chat locker 6 #9

Merged
merged 10 commits into from
Jul 17, 2015
Merged

Feature/chat locker 6 #9

merged 10 commits into from
Jul 17, 2015

Conversation

douglasrapp
Copy link

No description provided.

Douglas Rapp and others added 7 commits July 16, 2015 15:07
- Banner text and classification is retrieved from the server
- Added ReactiveDict package
- Added the AccessPermission class from previous app
- Included the as-yet non-existent 'accessPermissions' field to the list of chat room fields published to client
@@ -1748,11 +1748,24 @@ a.github-fork {
color: @primary-font-color;
}
}
.security-banner.U { color: white; background-color: green }
Copy link

Choose a reason for hiding this comment

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

maybe we can create variables for these in _variables.less and reference them here. Then we can reuse them in other places.

Copy link
Author

Choose a reason for hiding this comment

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

Do you mean variables for the colors? Like, say, "@unclassified-color"?

Copy link
Author

Choose a reason for hiding this comment

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

Done.

@rwakida
Copy link

rwakida commented Jul 17, 2015

Looks good

@douglasrapp
Copy link
Author

Merging to master

douglasrapp pushed a commit that referenced this pull request Jul 17, 2015
@douglasrapp douglasrapp merged commit 8dd5d1f into master Jul 17, 2015
@bbrockman bbrockman modified the milestone: Product Backlog Aug 3, 2015
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.

3 participants