Skip to content
This repository was archived by the owner on Jan 17, 2025. It is now read-only.

Fix 49: Missing 'license' at end #57

Closed
wants to merge 3 commits into from
Closed

Fix 49: Missing 'license' at end #57

wants to merge 3 commits into from

Conversation

@gabrielchl
Copy link
Contributor

@gabrielchl gabrielchl commented Mar 26, 2019

Fixes #49

Description

  • changed print_license_html() to include the text "License" as the name of CC licenses doesn't include the word "License".
  • removed the word "License" from the default license name
  • removed trailing spaces according to wordpress coding standards (I have no idea how those white spaces got there)

@kgodey kgodey requested a review from hugosolar March 26, 2019 19:46
hugosolar
hugosolar suggested changes Apr 3, 2019
View reviewed changes
Copy link
Contributor

@hugosolar hugosolar left a comment

Choose a reason for hiding this comment

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

The PR has merge conflicts. you should fix these conflicts and make the PR again

@gabrielchl gabrielchl closed this Apr 3, 2019
@gabrielchl gabrielchl deleted the fix-49 branch April 3, 2019 14:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@hugosolar hugosolar hugosolar requested changes

Reviewers whose approvals may not affect merge requirements

Assignees

@hugosolar hugosolar

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@gabrielchl @hugosolar @TimidRobot