Skip to content

Add method to make easier to add terms to post - #425

Open
jgrossi wants to merge 11 commits into
2.7from
add-term-to-post
Open

Add method to make easier to add terms to post#425
jgrossi wants to merge 11 commits into
2.7from
add-term-to-post

Conversation

@jgrossi

@jgrossi jgrossi commented Nov 13, 2018

Copy link
Copy Markdown
Member

This PR adds a addTerm(string $taxonomy, string $term) method to the Post model.

It partially fixes issue #410.

This way you don't have to worry about relations when adding terms to a given post:

$post = Post::find(1);
$term_instance = $post->addTerm('foo', 'Foo Term'); // "Foo Term" slug would be "foo-term"

🔥😎

@kushsharma

Copy link
Copy Markdown

Hi, can you pull this up? @jgrossi
The code looks fine other than a merge conflict as those helpers would be really useful.

@jgrossi

jgrossi commented May 17, 2019

Copy link
Copy Markdown
Member Author

hey @kushsharma gonna try to fix conflicts today or tomorrow, merge it and release a new version. probably gonna change the repo from 2.7 to 2.8, the current one. cheers

@izdrail

izdrail commented Sep 3, 2020

Copy link
Copy Markdown

any updates on this?

@marcpre

marcpre commented Dec 10, 2020

Copy link
Copy Markdown

Any update on this method?

@izdrail

izdrail commented Apr 9, 2021

Copy link
Copy Markdown

So why is this still pending
1 year ago I really needed this functionality into corcel
If anything I can fork it and fix the things.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants