Merged
Conversation
Owner
kashifpk
commented
Feb 10, 2025
kashifpk
commented
- Billing, notifications and overall restructuring
- Notifications API moved from core to notifications app. Test cases updates and better structuring.
- Billing plans limit checker with test cases
- Enforce billing limits on POST requests for resource creation
- Bigger tests fix after introducing plan based limits on product, project and component creation. This required fixing test cases across the whole project.
- Style fixes.
- Use proper error page when display billing limits related error.
- Ignore cursorrules folder as that is IDE specific and should not be placed in the repo.
- Ensure that sboms become public when downgrading to community plan.
- Test cases for core rename api
…ect and component creation. This required fixing test cases across the whole project.
…re cursorrules folder as that is IDE specific and should not be placed in the repo.
…dates and better strcturing.
…TRIPE env vars for ci/cd.
…a test environment inside the billing data migration. This attempts to fix that.
…probably the cause of the failing build due to migrations issue. This attempts to fix it.
…l patches, mocks and fixtures for billing settings, now they are managed in test_settings.py which is more proper. Replaced all billing fixtures with one unified fixture and ensured that it is used automatically to avoid making calls to stripe during tests by accident. Update the sample values for billing related fixtures to be more inline with what stripe actually returns.
… to fail in CI/CD
…lecting static files etc work during tests. This aims to fix the CI/CD issues.
… production flow for test cases.
… of sbomify.test_settings as the DJANGO_SETTINGS_MODULE. Hopefully this fixes the issue.
…h rights to the test databse to drop and create it.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.