Skip to content

[ BB2-1259 ] Update license#16

Merged
dtisza1 merged 1 commit intomainfrom
dtisza1/BB2-1259-update-license-file
Apr 26, 2022
Merged

[ BB2-1259 ] Update license#16
dtisza1 merged 1 commit intomainfrom
dtisza1/BB2-1259-update-license-file

Conversation

@dtisza1
Copy link
Copy Markdown
Contributor

@dtisza1 dtisza1 commented Apr 15, 2022

JIRA Ticket:
BB2-1259

User Story or Bug Summary:

This updates the license file to use the CMS standard Creative Commons Zero v1.0 Universal.

What Does This PR Do?

The following is a summary of the files in this PR

What Should Reviewers Watch For?

If you're reviewing this PR, please check these things, in particular:

  • Are there any unhandled and/or untested edge cases you can think of?

  • Does this make any backwards-incompatible changes that might break end user clients?

  • Can you find any bugs if you run the code locally and test it manually?

  • Naming in general? Do the namings of variables, methods, classes, and other items make sense? Are they too verbose, or not verbose enough? Any redundancies? Are the comments descriptive enough?

  • Are audit logging items properly handled? Is sensitive information either not included or hashed?

  • What/how to review:

@dtisza1 dtisza1 requested review from JFU-GIT and oragame April 15, 2022 16:29
@dtisza1 dtisza1 self-assigned this Apr 15, 2022
Copy link
Copy Markdown
Contributor

@oragame oragame 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!

Copy link
Copy Markdown
Contributor

@JFU-GIT JFU-GIT left a comment

Choose a reason for hiding this comment

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

Approved

@dtisza1 dtisza1 merged commit 8957cf3 into main Apr 26, 2022
@dtisza1 dtisza1 deleted the dtisza1/BB2-1259-update-license-file branch April 26, 2022 15:28
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