Skip to content

Eagle eye content wrong results with offset and limit#537

Merged
epipav merged 4 commits into
feature/eagle-eye-2from
bugfix/eagle-eye-content-nested-limit-offset
Feb 15, 2023
Merged

Eagle eye content wrong results with offset and limit#537
epipav merged 4 commits into
feature/eagle-eye-2from
bugfix/eagle-eye-content-nested-limit-offset

Conversation

@epipav
Copy link
Copy Markdown
Collaborator

@epipav epipav commented Feb 15, 2023

Changes proposed ✍️

  • fixes wrong results when limit and offset is provided

Checklist ✅

  • Label appropriately with Feature, Enhancement, or Bug.
  • Tests are passing.
  • New backend functionality has been unit-tested.
  • Environment variables have been updated:
    • Local frontend configuration: frontend/.env.dist.local, frontend/.env.dist.composed.
    • Local backend: backend/.env.dist.local, backend/.env.dist.composed.
    • Configuration docs have been updated.
    • Team members only: update environment variables in override, staging and production env. files and trigger update config script.
  • API documentation has been updated (if necessary) (see docs on API documentation).
  • Quality standards are met.
  • All changes have been tested in a staging site.
  • All changes are working locally running crowd.dev's Docker local environment.

@epipav epipav changed the base branch from main to feature/eagle-eye-2 February 15, 2023 15:22
@epipav epipav self-assigned this Feb 15, 2023
@epipav epipav added the Bug Created by Linear-GitHub Sync label Feb 15, 2023
@epipav epipav marked this pull request as ready for review February 15, 2023 15:24
@epipav epipav requested a review from joanreyero February 15, 2023 16:21
Copy link
Copy Markdown
Contributor

@joanreyero joanreyero left a comment

Choose a reason for hiding this comment

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

Thanks for handling this!

@epipav epipav merged commit 5a4a08b into feature/eagle-eye-2 Feb 15, 2023
@epipav epipav deleted the bugfix/eagle-eye-content-nested-limit-offset branch February 15, 2023 17:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Created by Linear-GitHub Sync

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants