Skip to content

feat(public-api): add listId support to POST /bookmarks#3546

Closed
idoshamun wants to merge 1 commit intomainfrom
feat/public-api-bookmark-to-list
Closed

feat(public-api): add listId support to POST /bookmarks#3546
idoshamun wants to merge 1 commit intomainfrom
feat/public-api-bookmark-to-list

Conversation

@idoshamun
Copy link
Member

Summary

Adds the ability to bookmark posts to a specific list via the public API.

Changes

  • GraphQL: Added optional listId field to AddBookmarkInput
  • Resolver: Validates Plus subscription and list ownership when listId is provided
  • Public API: POST /bookmarks now accepts optional listId in request body
  • Response: Returns list info (id, name) in bookmark response

Usage

# Bookmark to default (last used) list
curl -X POST /v1/bookmarks -d '{"postIds": ["abc123"]}'

# Bookmark to specific list (Plus feature)
curl -X POST /v1/bookmarks -d '{"postIds": ["abc123"], "listId": "list-uuid"}'

Notes

  • Requires Plus subscription when listId is specified
  • Returns 403 if user is not Plus or doesn't own the list
  • Test suite has pre-existing zod schema issue unrelated to these changes

- Add optional listId parameter to AddBookmarkInput GraphQL type
- Validate user is Plus and owns list when listId is provided
- Update public API endpoint to accept listId in request body
- Return list info in bookmark response
- Add comprehensive tests for listId functionality
@pulumi
Copy link

pulumi bot commented Feb 5, 2026

🍹 The Update (preview) for dailydotdev/api/prod (at 2b6422c) was successful.

Resource Changes

    Name                                                       Type                           Operation
~   vpc-native-generate-search-invites-cron                    kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-updated-sync-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-generic-referral-reminder-cron                  kubernetes:batch/v1:CronJob    update
~   vpc-native-user-posts-analytics-refresh-cron               kubernetes:batch/v1:CronJob    update
~   vpc-native-personalized-digest-deployment                  kubernetes:apps/v1:Deployment  update
~   vpc-native-update-source-tag-view-cron                     kubernetes:batch/v1:CronJob    update
~   vpc-native-update-trending-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-user-companies-cron                kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-gifted-plus-cron                          kubernetes:batch/v1:CronJob    update
~   vpc-native-sync-subscription-with-cio-cron                 kubernetes:batch/v1:CronJob    update
~   vpc-native-validate-active-users-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-analytics-history-clickhouse-cron  kubernetes:batch/v1:CronJob    update
-   vpc-native-api-clickhouse-migration-abbdec1e               kubernetes:batch/v1:Job        delete
~   vpc-native-daily-digest-cron                               kubernetes:batch/v1:CronJob    update
-   vpc-native-api-db-migration-abbdec1e                       kubernetes:batch/v1:Job        delete
~   vpc-native-post-analytics-history-day-clickhouse-cron      kubernetes:batch/v1:CronJob    update
~   vpc-native-temporal-deployment                             kubernetes:apps/v1:Deployment  update
~   vpc-native-update-views-cron                               kubernetes:batch/v1:CronJob    update
~   vpc-native-calculate-top-readers-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-hourly-notification-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-users-cron                         kubernetes:batch/v1:CronJob    update
~   vpc-native-update-highlighted-views-cron                   kubernetes:batch/v1:CronJob    update
~   vpc-native-bg-deployment                                   kubernetes:apps/v1:Deployment  update
~   vpc-native-ws-deployment                                   kubernetes:apps/v1:Deployment  update
+   vpc-native-api-clickhouse-migration-fca76df1               kubernetes:batch/v1:Job        create
~   vpc-native-check-analytics-report-cron                     kubernetes:batch/v1:CronJob    update
~   vpc-native-deployment                                      kubernetes:apps/v1:Deployment  update
+   vpc-native-api-db-migration-fca76df1                       kubernetes:batch/v1:Job        create
~   vpc-native-personalized-digest-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-user-profile-analytics-clickhouse-cron          kubernetes:batch/v1:CronJob    update
~   vpc-native-private-deployment                              kubernetes:apps/v1:Deployment  update
~   vpc-native-update-source-public-threshold-cron             kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tags-str-cron                            kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-images-cron                        kubernetes:batch/v1:CronJob    update
~   vpc-native-update-current-streak-cron                      kubernetes:batch/v1:CronJob    update
~   vpc-native-update-tag-recommendations-cron                 kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-stale-user-transactions-cron              kubernetes:batch/v1:CronJob    update
~   vpc-native-clean-zombie-opportunities-cron                 kubernetes:batch/v1:CronJob    update
~   vpc-native-expire-super-agent-trial-cron                   kubernetes:batch/v1:CronJob    update
~   vpc-native-post-analytics-clickhouse-cron                  kubernetes:batch/v1:CronJob    update
... and 1 other changes

@idoshamun idoshamun closed this Feb 5, 2026
@idoshamun idoshamun deleted the feat/public-api-bookmark-to-list branch February 5, 2026 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant