- 
          
- 
                Notifications
    You must be signed in to change notification settings 
- Fork 1.7k
feat: Delete store endpoint code #4969
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
Conversation
| size-limit report 📦
 | 
817a118    to
    4e64c8d      
    Compare
  
    f8d5392    to
    88eac76      
    Compare
  
    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.
Looks good. A general concern of mine is that the envelope file is completely untested.
| /** | ||
| * Create an Envelope from an event. | ||
| */ | ||
| export function createEventEnvelope( | 
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.
We should def write tests for the functions in this file. Don't know whether this PR is the place for this tho. Wdyt?
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.
Yes, we should write tests - I can write up a ticket after this gets merged in.
5ad27a4    to
    dac51b7      
    Compare
  
    88eac76    to
    6d3043a      
    Compare
  
    
Delete all code related to store endpoint.
Built on top of work in #4967
Resolves https://getsentry.atlassian.net/browse/WEB-557