Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make a trait out of PaginationCursor / PaginationCursorData #5275

Open
5 tasks done
dessalines opened this issue Dec 19, 2024 · 1 comment · May be fixed by #5424
Open
5 tasks done

Make a trait out of PaginationCursor / PaginationCursorData #5275

dessalines opened this issue Dec 19, 2024 · 1 comment · May be fixed by #5424
Labels
area: maintenance enhancement New feature or request
Milestone

Comments

@dessalines
Copy link
Member

dessalines commented Dec 19, 2024

Requirements

  • Is this a feature request? For questions or discussions use https://lemmy.ml/c/lemmy_support
  • Did you check to see if this issue already exists?
  • Is this only a feature request? Do not put multiple feature requests in one issue.
  • Is this a backend issue? Use the lemmy-ui repo for UI / frontend issues.
  • Do you agree to follow the rules in our Code of Conduct?

Is your proposal related to a problem?

We currently have a lot of custom PaginationCursors (PostView, all the new Combined endpoints that now use pagination cursors) , with the same after_post and read functions.

Describe the solution you'd like.

This should be made into a trait that each one can implement.

Also, see this comment about making a helper function for ids / prefixes

Describe alternatives you've considered.

NA

Additional context

NA

@dessalines
Copy link
Member Author

I'll take a look at doing this as part of the 1.0.0 milestone. If it doesn't work out I'll just close this.

@dessalines dessalines added this to the 1.0.0 milestone Feb 12, 2025
@dessalines dessalines linked a pull request Feb 13, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: maintenance enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant