Skip to content

Geo Package Files - GraphQL Pagination

Zack Cuddy requested to merge 208470_04-package-file-graphql-pagination into master

What does this MR do?

Officially closes #208470 (closed)

This MR splits off from !32872 (closed)

This MR is an attempt at MVC to avoid the above massive MR going in all at once.

This MR hooks up the GraphQL pagination logic required to work along side of the existing RESTful pagination logic.

This is done via the GeoGraphqlPagination component.
This change also removes the feature flag needed for this feature and adds a changelog

Important note about ~"technical debt"

The logic between the two pagination types will be removed as part of this follow up issue: #219417 (closed)

GIF

Pagination: Kapture_2020-06-05_at_16.05.33

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • [-] Label as security and @ mention @gitlab-com/gl-security/appsec
  • [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Zack Cuddy

Merge request reports