Skip to content

Fix #167: add JsonLd::setName() - #168

Open
klimov-paul wants to merge 7 commits into
artesaos:masterfrom
klimov-paul:167-json-ld-set-name
Open

Fix #167: add JsonLd::setName()#168
klimov-paul wants to merge 7 commits into
artesaos:masterfrom
klimov-paul:167-json-ld-set-name

Conversation

@klimov-paul

Copy link
Copy Markdown
Contributor
Q A
Is bugfix?
New feature? ✔️
Breaks BC?
Tests pass? ✔️
Fixed issues #167

@J-Brk

J-Brk commented Nov 24, 2021

Copy link
Copy Markdown
Collaborator

@vinicius73 Can this be merged?

@vinicius73

Copy link
Copy Markdown
Member

There are conflicts to solve @J-Brk
Can you fix it?

@J-Brk

J-Brk commented Nov 29, 2021

Copy link
Copy Markdown
Collaborator

@vinicius73 I'm sorry, I missed to notice there was any conflict. Unfortunately it's not showing me what the conflicts are. Can you tell me what the conflict is with this pull request?

image

@J-Brk

J-Brk commented Feb 12, 2022

Copy link
Copy Markdown
Collaborator

Sorry for the individual commits; I was quickly doing it from the web editor but turns out there was just too much missing.

Actually it would have been nicer to make setName fall back to setTitle for the consistency; since all other functions are already called setTitle.

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