Proposal/pagination

[Nisarg-Chokshi]

Problem

👋 Thanks for opening a pull request! Please include a brief summary of the problem your change is trying to solve, or bug fix. If your change fixes a bug or you'd like to provide context on why you're making the change, please link the issue as follows:

• The Arrow buttons of pagination were displayed even if there is no content available for Datasets/Events

Solution

Please describe your change as it relates to the problem, or bug fix, as well as any dependencies. If your change requires a database schema migration, please describe the schema modification(s) and whether it's a backwards-incompatible or backwards-compatible change.

Note: All database schema changes require discussion. Please link the issue for context.

One-line summary: Updated Dataset & Event route render to remove pagination in case of no content

Checklist

• You've signed-off your work
• Your changes are accompanied by tests (if relevant)
• Your change contains a small diff and is self-contained
• You've updated any relevant documentation (if relevant)
• You've included a one-line summary of your change for the CHANGELOG.md (Depending on the change, this may not be necessary).
• You've versioned your .sql database schema migration according to Flyway's naming convention (if relevant)
• You've included a header in any source code files (if relevant)

Snapshots

[codecov]

Codecov Report

Merging #2559 (81abe21) into main (e14cf9b) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main    #2559   +/-   ##
=========================================
  Coverage     83.26%   83.26%           
  Complexity     1286     1286           
=========================================
  Files           243      243           
  Lines          5934     5934           
  Branches        279      279           
=========================================
  Hits           4941     4941           
  Misses          846      846           
  Partials        147      147           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[wslulciuc]

LGTM 👍

[phixMe]

I would use the && notation here since you want to return a null value instead of an empty logical tag.

[NisargChokshi45]

@phixMe I've added a new commit addressing the change! Could you please have a quick review of it?

[phixMe]

Thanks