Skip to content

DEV-427-Add subscription type for twitter user get request - #7

Merged
saflukeahmed merged 1 commit into
mainfrom
DEV-427-Add-Subscription-type-to-get-user-request
Aug 6, 2026
Merged

DEV-427-Add subscription type for twitter user get request#7
saflukeahmed merged 1 commit into
mainfrom
DEV-427-Add-Subscription-type-to-get-user-request

Conversation

@saflukeahmed

Copy link
Copy Markdown

DEV-427

Description of Changes
Add a cache for Twitter Subscription types

Documentation
n/a

Risks & Impacts
n/a

Security
n/a

Testing
Tested Locally

Deployment Considerations
https://echobox.atlassian.net/browse/DEV-427

@saflukeahmed
saflukeahmed requested a balanced review from Copilot August 6, 2026 21:30

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Warning

Copilot couldn't run its full agentic review because it didn't start before the timeout. Make sure your repository has a runner available, or add a copilot-code-review.yml file specifying one with the runs-on attribute. See the docs for more details.

Adds support for a Twitter/X user subscription_type attribute by extending the existing User model and ensuring it participates in serialization and object identity methods.

Changes:

  • Added subscriptionType field with Gson @SerializedName mapping to subscription_type
  • Added getter/setter and fluent setter for subscriptionType
  • Updated equals, hashCode, toString, and openapiFields to include subscription_type

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/java/com/twitter/clientlib/model/User.java
Comment thread src/main/java/com/twitter/clientlib/model/User.java
@saflukeahmed saflukeahmed changed the title DEV-374-Add subscription type for twitter user get request DEV-427-Add subscription type for twitter user get request Aug 6, 2026
@saflukeahmed
saflukeahmed merged commit 717485c into main Aug 6, 2026
1 check was pending
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.

2 participants