Integrating InfiniteScroll to the Discovery view #39

Merged
anomalist merged 2 commits from discover-infinite-scroll into master 1 year ago
anomalist commented 1 year ago
Owner
There is no content yet.
anomalist added 1 commit 1 year ago
trentlarson reviewed 1 year ago
const plans: ProjectData[] = results.data;
for (const plan of plans) {
const { name, description, handleId = plan.fullIri, rowid } = plan;
if (beforeId !== plan["rowid"]) {
Poster
Owner

My apologies for this! I see that the server code does NOT exclude the "beforeId" and "afterId" on this endpoint. I will fix that.

My apologies for this! I see that the server code does NOT exclude the "beforeId" and "afterId" on this endpoint. I will fix that.
Owner

It's true that the results will be different: the location-based search will only find projects which have been given a location and which lie within the bounds sent in the request. Let's talk if that doesn't make sense.

It's true that the results will be different: the location-based search will only find projects which have been given a location and which lie within the bounds sent in the request. Let's talk if that doesn't make sense.
anomalist added 1 commit 1 year ago
anomalist merged commit 00954693b5 into master 1 year ago
The pull request has been merged as 00954693b5.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.