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

Add table and logic to store discouraged details #1211

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jcscottiii
Copy link
Collaborator

This change adds a new table: FeatureDiscouragedDetails

It is setup to have a foreign key with its corresponding Web Feature.

The other fields come from web features.

We don't plan on querying for individual details in AccordingTo and Alternatives. Those two fields are marked as nullable because we may get rid of them in the future.

jcscottiii added a commit that referenced this pull request Feb 25, 2025
This change modifies the web features workflow to call the spanner logic
that inserts discouraged details (if those details exist for a feature)

Builds on #1211
This change adds a new table: FeatureDiscouragedDetails

It is setup to have a foreign key with its corresponding  Web Feature.

The other fields come from [web features](https://github.com/web-platform-dx/web-features/blob/1332db1a8ceb74af13fbcb58ee8501aa2b948f71/schemas/data.schema.json#L60-L81).

We don't plan on querying for individual details in AccordingTo and Alternatives.
Those two fields are marked as nullable because we may get rid of them in the future.
@jcscottiii jcscottiii force-pushed the jcscottiii/store-discouraged-details-spanner branch from f236bc8 to afca1b8 Compare February 25, 2025 21:05
jcscottiii added a commit that referenced this pull request Feb 25, 2025
This change modifies the web features workflow to call the spanner logic
that inserts discouraged details (if those details exist for a feature)

Builds on #1211
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.

1 participant