Skip to content

Conversation

@ArnavBallinCode
Copy link
Contributor

After PR #1258 fixed the export page HTTP 500 error, This pr fixes issue #1282 , export functionality which was broken for some of the exporters

Issues found and fixed

  • Code still referenced old item field instead of new product field
  • JSON_FIELD_AVAILABLE setting was undefined
  • Form data accessed without defaults
  • Invalid Query Syntax**: Wrong Django filter methods
  • Empty Data Handling**: No safeguards for reports with no data

Related Issues

Copy link
Contributor

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @ArnavBallinCode, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

Copy link
Member

@mariobehling mariobehling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please resolve conflicts.

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.

Export Functionality Broken

2 participants