-
Notifications
You must be signed in to change notification settings - Fork 30
Add package listing report feature #1062
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
x753
wants to merge
23
commits into
master
Choose a base branch
from
PackageReport
base: master
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
Conversation
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
Roffenlund
reviewed
Jan 21, 2025
Roffenlund
reviewed
Jan 21, 2025
Roffenlund
reviewed
Jan 21, 2025
Roffenlund
reviewed
Jan 21, 2025
django/thunderstore/community/api/experimental/views/listing.py
Outdated
Show resolved
Hide resolved
Roffenlund
reviewed
Jan 21, 2025
Roffenlund
reviewed
Jan 21, 2025
Add ts_reports app Add PackageReport model Add PackageReport admin
Create an API for reporting PackageListings (creating a PackageReport)
Adds a report button to the package details page Adds the react / tsx necessary to make it work Modifies APIError to take an extractedMessage Renames BaseApiError to BaseValidationError Adds the "required" optional property to FormSelectFieldProps
Add tests for the Package Listing Report API and serializer
Create a Cyberstorm Package Listing Report API for future use.
Roffenlund
reviewed
Jan 30, 2025
Add tests for Cyberstorm listing report API
Roffenlund
reviewed
Feb 7, 2025
Refactor report_button_props into a separate function because get_context_data was becoming too large
Roffenlund
reviewed
Feb 11, 2025
Roffenlund
reviewed
Feb 11, 2025
Roffenlund
reviewed
Feb 11, 2025
Roffenlund
reviewed
Feb 11, 2025
Roffenlund
reviewed
Feb 11, 2025
Remove Cyberstorm PackageReport API, to be later implemented after package listing action views have been implemented or otherwise necessary.
Update PackageReport request serializer to use PrimaryKeyRelatedField instead of ModelChoiceField to handle errors better
Roffenlund
approved these changes
Feb 12, 2025
Add ts_reports app Add PackageReport model Add PackageReport admin
Create an API for reporting PackageListings (creating a PackageReport)
Adds a report button to the package details page Adds the react / tsx necessary to make it work Modifies APIError to take an extractedMessage Renames BaseApiError to BaseValidationError Adds the "required" optional property to FormSelectFieldProps
Add tests for the Package Listing Report API and serializer
Create a Cyberstorm Package Listing Report API for future use. # Conflicts: # django/thunderstore/api/cyberstorm/views/__init__.py
Add tests for Cyberstorm listing report API
Refactor report_button_props into a separate function because get_context_data was becoming too large
Remove Cyberstorm PackageReport API, to be later implemented after package listing action views have been implemented or otherwise necessary.
Update PackageReport request serializer to use PrimaryKeyRelatedField instead of ModelChoiceField to handle errors better
…understore into PackageReport
Remove leftover reference to Cyberstorm ReportListingAPIView
Fix typo in PackageReportAdmin, thanks graphite bot
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds the PackageReport model, as well as an API for sending reports for specific listings and a button / react component to use said API.
This is heavily based on this branch: https://github.com/thunderstore-io/Thunderstore/tree/report-feature
firefox_PafGD8xsH4.mp4