Skip to content

Plugin support for fields tags #389

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
Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Plugin support for fields tags #389

wants to merge 2 commits into from

Conversation

@bc-tayvs
Copy link

@bc-tayvs bc-tayvs commented Jul 14, 2025 •
edited
Loading

Issue addressed in this PR: #390

I have:

  • Written a clear PR title and description (above)
  • Signed the Khan Academy CLA
  • Added tests covering my changes, if applicable
  • Included a link to the issue fixed, if applicable
  • Included documentation, for new features
  • Added an entry to the changelog

@bc-tayvs bc-tayvs mentioned this pull request Jul 14, 2025
Open
@bc-tayvs bc-tayvs changed the title plugin support for fields tags Plugin support for fields tags Jul 14, 2025
@bc-tayvs bc-tayvs marked this pull request as ready for review July 23, 2025 13:51
@benjaminjkraft
Copy link
Collaborator

benjaminjkraft commented Aug 2, 2025

Thanks -- let's discuss on #390/#237 as defining the right API we can stick to is the main thing here.

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

Reviewers

@benjaminjkraft benjaminjkraft Awaiting requested review from benjaminjkraft benjaminjkraft is a code owner

@csilvers csilvers Awaiting requested review from csilvers csilvers is a code owner

@dnerdy dnerdy Awaiting requested review from dnerdy dnerdy is a code owner

@StevenACoffman StevenACoffman Awaiting requested review from StevenACoffman StevenACoffman is a code owner

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bc-tayvs @benjaminjkraft