Skip to content

chore: change filtering#780

Draft
peter-hammans wants to merge 1 commit intodevfrom
fix/collections-in-team-summary
Draft

chore: change filtering#780
peter-hammans wants to merge 1 commit intodevfrom
fix/collections-in-team-summary

Conversation

@peter-hammans
Copy link
Collaborator

@peter-hammans peter-hammans commented Oct 10, 2024

Screenshots (if relevant)

Describe your changes

Looks like toArray doesn't always work when querying the collections for a team. Sometimes it creates an associative array with the indices as keys. This PR creates an array a different way.

Issue ticket link

Environment / Configuration changes (if applicable)

Requires migrations being run?

If not using the pre-push hook. Confirm tests pass:

Checklist before requesting a review

  • I have performed a self-review of my code
  • I have added appropriate unit tests
  • I have created mocks for unit tests (where appropriate)
  • I have added appropriate Behat tests to confirm AC (if applicable)
  • I have added Swagger annotations for new endpoints (if applicable)
  • I have added audit logs for new operation logic (if applicable)
  • I have added new environment variables to the .env.example file (if applicable)
  • I have added new environment variables to terraform repository (if applicable)

@peter-hammans peter-hammans marked this pull request as draft October 10, 2024 08:21
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.

1 participant