Skip to content
This repository was archived by the owner on Apr 5, 2023. It is now read-only.

Comments

Add asset ID plugin#4

Open
AndrewGibson27 wants to merge 20 commits intomasterfrom
asset-id
Open

Add asset ID plugin#4
AndrewGibson27 wants to merge 20 commits intomasterfrom
asset-id

Conversation

@AndrewGibson27
Copy link

What does this PR do?

Adds this plugin to our build.

This branch is merged into the test branch, and I've started sending asset_id at render time in texastribune's auth branch. This means you can verify everything is working by going to https://tt-auth-test.texastribune.org and adding a comments plugin to an article that doesn't already have one (I get a GraphQL error when I render an embed that was created before this plugin was added. I think that makes sense.)

Note this branch is itself a branch off auth0, so we should merge the latter first.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants