-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Monthly shareable link quota and free-plan duration limits #1800
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
Open
richiemcilroy
wants to merge
36
commits into
main
Choose a base branch
from
quota-limits
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
aefe0e6
feat(database): add index for owner non-screenshot video counts
richiemcilroy 4158d11
feat(web-domain): add shareable link usage and limit error schemas
richiemcilroy f296f09
feat(web-domain): add shareable link limit to Loom HTTP and workflow
richiemcilroy 1629a4d
feat(web-backend): add shareable link monthly quota helpers
richiemcilroy 4f62d0c
feat(web-backend): enforce shareable link quota on video insert
richiemcilroy 76efd38
feat(web-backend): map loom create activity to shareable link limit
richiemcilroy 6a570a4
feat(web): add UTC month day ordinal date formatter
richiemcilroy 923a6ea
feat(web): return monthly shareable link usage from billing API
richiemcilroy bfc1908
feat(web): load shareable link usage in dashboard layout
richiemcilroy 545c79f
feat(web): show monthly share link quota in usage sidebar
richiemcilroy 0afa11a
feat(web): enforce shareable link rules in desktop video API
richiemcilroy 97a8ddc
feat(web): enforce shareable link limits on upload routes
richiemcilroy 17c11d8
feat(web): validate free duration on media server progress complete
richiemcilroy 9ce4a17
feat(web): enforce shareable quota in video server actions
richiemcilroy 2b7f130
feat(web): enforce shareable quota in Loom import action
richiemcilroy e5b2ab9
feat(web): block over-limit durations in processing workflows
richiemcilroy bcb5656
feat(web): show shareable link usage on share page for owners
richiemcilroy 94dd2dc
style(web): sentence-case upgrade CTA labels
richiemcilroy 87b68f6
feat(web): toast shareable link limit in web recorder
richiemcilroy ce50fd1
test(web): mock shareable link quota in loom import test
richiemcilroy 706b080
test(web): add shareable link usage helper tests
richiemcilroy 2251dbe
feat(desktop): track plan shareable usage and drop upgrade window
richiemcilroy 2f31950
feat(desktop): surface share link quota and open pricing externally
richiemcilroy 974b129
feat(web-backend): add markShareableLinkUploadRejected helper
richiemcilroy 4f7e2ec
fix(web-backend): detect shareable limits on nested Error causes
richiemcilroy f3299e6
fix(web-backend): skip error-phase uploads in shareable quotas
richiemcilroy 0078056
test(web): cover wrapped shareable link limit errors
richiemcilroy 480bede
fix(desktop): allow only http/https upgrade base URLs
richiemcilroy 33627fb
fix(settings): expose true video totals in billing usage
richiemcilroy 8709954
fix(webhooks): centralize quota rejection and reply 200 on limits
richiemcilroy 9639e2b
refactor(web): unify shareable quota rejection in workflows
richiemcilroy 81ed85e
feat(web): persist quota rejects when creating uploads
richiemcilroy 160da3b
fix(upload): honor screenshot uploads in multipart quota check
richiemcilroy 46d87c3
feat(upload): persist quota rejects on multipart completion
richiemcilroy a4fa658
feat(upload): mark quota rejects on recording-complete failures
richiemcilroy ccf2838
feat(upload): mark quota rejects on signed upload failures
richiemcilroy File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since
server_urlcan come from user settings, it may be worth validating the scheme before building/launching the URL (e.g. allowhttp:///https://only) so we don’t accidentally open arbitrary schemes if someone misconfiguresserver_url.