Skip to content

Classification#2

Open
clabas wants to merge 48 commits into
ozoneplatform:masterfrom
nssbu:classification
Open

Classification#2
clabas wants to merge 48 commits into
ozoneplatform:masterfrom
nssbu:classification

Conversation

@clabas

@clabas clabas commented Dec 19, 2016

Copy link
Copy Markdown

This pull request adds support for dynamically setting classification banners based on an overall system classification. Per Igor, despite this technically being a breaking change, making this effort available to the community.

Note that this contains the changes added in the 'cas' branch.

akonsowski and others added 30 commits September 7, 2016 13:51
fix(index):some menu options will not show up for second party users
fix(index):added a check to see if we should display webtop
feat(order): Introduced Ordering of Icon #231
Changed the Share New Folder button text to say Back when on the Send…
…ogin screen. webpack dev server now serves at :8088/ instead of :8088/dist Various URL repairs in package.json and upgraded ozp-react-commons.
Charles Labas and others added 18 commits December 19, 2016 11:33
…ication banners configurable"

This reverts commit 020c38a.
@mparizernc

Copy link
Copy Markdown

@clabas we appreciate your pull request and would like to put it forward for our GOSS Board to determine if this should be included in the product. We have just formalized our patch process that can be found at https://ozone.nextcentury.com/patch_process. Please ensure that the following has been included so that it can be presented to the board:

  • Add comments in the code to provide detailed developer-level documentation of your changes
  • Verify that the entire code base builds without issue after all changes are made
  • Document and provide unit/regression tests that verify that your new code functions and can be tested by others
  • Submit complete and adequate documentation with the pull request explaining the feature(s) and how it works

Please be sure to include in your documentation how this will be a breaking change.

Thank you again for your contribution to this project.

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.

8 participants