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

feat: audience filters via jwt #1629

Merged
merged 4 commits into from
Oct 10, 2024
Merged

Conversation

abvthecity
Copy link
Contributor

@abvthecity abvthecity commented Oct 10, 2024

Updates the pruning logic introduced in #1627 to match on audiences while remaining backwards compatible with route-matching.

Based on the spec laid out here: https://buildwithfern.notion.site/Scoping-Audiences-in-Fern-Docs-11ab23082e4c80b09eaac8a9eca15a16?pvs=74

@abvthecity abvthecity marked this pull request as ready for review October 10, 2024 05:08
Copy link

Copy link

@abvthecity abvthecity had a problem deploying to Preview - app-dev.buildwithfern.com October 10, 2024 05:53 — with GitHub Actions Error
@abvthecity abvthecity had a problem deploying to Preview - app.buildwithfern.com October 10, 2024 05:53 — with GitHub Actions Error
Copy link

@abvthecity abvthecity had a problem deploying to Preview - app.buildwithfern.com October 10, 2024 05:55 — with GitHub Actions Failure
@abvthecity abvthecity had a problem deploying to Preview - app-dev.buildwithfern.com October 10, 2024 05:55 — with GitHub Actions Failure
Copy link

Copy link

Playwright test results

passed  1 passed

Details

stats  1 test across 1 suite
duration  9 seconds
commit  37dfa23

Copy link

github-actions bot commented Oct 10, 2024

PR Preview

Copy link

github-actions bot commented Oct 10, 2024

Playwright test results

passed  81 passed

Details

stats  81 tests across 3 suites
duration  1 minute, 10 seconds
commit  37dfa23

@abvthecity abvthecity merged commit 20466fb into ajiang/mark-authed Oct 10, 2024
21 of 22 checks passed
@abvthecity abvthecity deleted the ajiang/jwt-audiences branch October 10, 2024 12:46
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.

2 participants