Skip to content

feat(types): map additional comment fields#1669

Merged
nbartels merged 1 commit into
devfrom
feat/comment-fields
May 7, 2026
Merged

feat(types): map additional comment fields#1669
nbartels merged 1 commit into
devfrom
feat/comment-fields

Conversation

@nbartels

@nbartels nbartels commented May 7, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Contributor

Summary

  • Add missing Graph API Comment fields for admin creator, application, private status, and live broadcast timestamp.
  • Map live_broadcast_timestamp as java.util.Date so existing mapper conversion handles the value.

Verification

  • mvn -DskipTests compile
  • mvn test

Summary by CodeRabbit

New Features

  • Comments now support additional fields: admin creator user information, associated application details, privacy status indicators, and live broadcast timestamp data for more comprehensive comment management and metadata access.

@coderabbitai

coderabbitai Bot commented May 7, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack
No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: de4457cc-8ea5-4cc7-984c-82d66a94f991

📥 Commits

Reviewing files that changed from the base of the PR and between cc56594 and c277748.

📒 Files selected for processing (1)
  • src/main/lombok/com/restfb/types/Comment.java
📝 Walkthrough

Walkthrough

The Comment class in the restfb library is extended with four new Graph API-mapped fields: adminCreator (User), application (Application), isPrivate (Boolean), and liveBroadcastTimestamp (Date). Each field is annotated with the corresponding Facebook Graph API attribute name.

Changes

Comment API Field Extensions

Layer / File(s) Summary
Data Model Extension
src/main/lombok/com/restfb/types/Comment.java		 Four new fields are added to Comment: adminCreator (User, mapped admin_creator), application (Application), isPrivate (Boolean, mapped is_private), and liveBroadcastTimestamp (Date, mapped live_broadcast_timestamp).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Four fields hop into Comment's fold,
Admin, app, and timestamps bold—
Private flags and broadcasts bloom,
Graph API finds more room!

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat(types): map additional comment fields' clearly and concisely summarizes the main change: adding new field mappings to the Comment type.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/comment-fields

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nbartels nbartels added this to the 2026.6.0 milestone May 7, 2026
@nbartels nbartels self-assigned this May 7, 2026
@nbartels
nbartels merged commit be9197c into dev May 7, 2026
4 checks passed
@nbartels
nbartels deleted the feat/comment-fields branch May 7, 2026 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@nbartels nbartels

Labels

None yet

Projects

None yet

Milestone

2026.6.0

Development

Successfully merging this pull request may close these issues.

1 participant

@nbartels