Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Slug for tags #1488

Merged
merged 1 commit into from
Dec 28, 2015
Merged

Slug for tags #1488

merged 1 commit into from
Dec 28, 2015

Conversation

nicosomb
Copy link
Member

No description provided.

@j0k3r
Copy link
Member

j0k3r commented Oct 14, 2015

Why do we need this? Are you going to use it somewhere? If yes, why don't you do that in this PR? :)

@nicosomb
Copy link
Member Author

I will use it in API (#1432): to fetch entries for a specific tag.

@j0k3r
Copy link
Member

j0k3r commented Oct 14, 2015

So maybe you can add the api endpoint right in this PR

@nicosomb nicosomb changed the title add slug for tags [WIP] add slug for tags Oct 15, 2015
@nicosomb nicosomb changed the title [WIP] add slug for tags [WIP] slug for tags / API endpoint for tags Oct 15, 2015
@j0k3r
Copy link
Member

j0k3r commented Oct 31, 2015

What left to be done here? Do you need help?

@j0k3r
Copy link
Member

j0k3r commented Dec 28, 2015

Updated & rebased.

@j0k3r
Copy link
Member

j0k3r commented Dec 28, 2015

@nicosomb I think we can merge that and check later for the missing endpoint.

@j0k3r j0k3r changed the title [WIP] slug for tags / API endpoint for tags Slug for tags / API endpoint for tags Dec 28, 2015
@j0k3r j0k3r changed the title Slug for tags / API endpoint for tags Slug for tags Dec 28, 2015
j0k3r added a commit that referenced this pull request Dec 28, 2015
@j0k3r j0k3r merged commit 82899c0 into v2 Dec 28, 2015
@j0k3r j0k3r deleted the v2-slug-tags branch December 28, 2015 12:12
@nicosomb
Copy link
Member Author

👍

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.

None yet

2 participants