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.

Sign up for GitHub

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

Jump to bottom

add content_translation to taxonomy term #425

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

add content_translation to taxonomy term #425

wants to merge 2 commits into from

Conversation

mikegikas
Copy link

No description provided.

Mike Gikas added 2 commits January 20, 2023 13:38
Update next_extras.module
f554f7d 
add content_translation to taxonomy terms
Update next_extras.module
bbeda9c
@vercel
Copy link

vercel bot commented Jan 20, 2023

Someone is attempting to deploy a commit to the Chapter Three Team on Vercel.

A member of the Team first needs to authorize it.

@p0zi p0zi mentioned this pull request Mar 9, 2023
Closed
@JohnAlbin
Copy link
Collaborator

This issue needs explanation. Please open a new issue and then link this PR to that issue.

@JohnAlbin JohnAlbin added area: next module need more info Maintainers need more info labels Oct 22, 2023
@JohnAlbin
Copy link
Collaborator

This PR is blocked until I understand what it does. The change looks small but the description doesn't tell anyone what the change does and I can't commit it without explaining what feature it adds.

@p0zi
Copy link

p0zi commented Feb 22, 2024

This adds to JSON:API response computed field "content_translations" also to taxonomy term entities.
This helps in frontends while language switching to have contextual translations links for other languages. Please see attached screens for explanation:

Until now we have this computed field only on "node" type id:
Screenshot from 2024-02-22 11-46-49

After change it appears also for taxonomy terms:
Screenshot from 2024-02-22 11-48-26

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

@nikoskokkalidis nikoskokkalidis nikoskokkalidis approved these changes

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area: next module need more info Maintainers need more info
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mikegikas @JohnAlbin @p0zi @nikoskokkalidis