Wrong `UsersShowParams` interface

The current UsersShowParams is wrong:

export interface UsersShowParams {
    /**
    * The ID of the user for whom to return results. Either an id or
  screen_name is required for this method.
  
    */
    user_id: string | number;
    /**
    * The screen name of the user for whom to return results. Either
  a id or screen_name is required for this method.
  
    */
    screen_name: string;
    /**
    * The entities node will not be included when set to false.
    */
    include_entities?: boolean;
}

It says user_id or screen_name is required but both are currently required.

[SimonHoiberg]

Hi @iliran11 Yeah, so it’s an unfortunate occasion because both parameters are flagged as required but in the description, it actually says “Either an id or screen_name is required for this method”.

So it means that you’re supposed to provide either and id or a screen_name parameter. But in the twitter-api-spec.yml, these are both set to required, which is what the UsersShowParams interface is based on.

There’s a way to make a type in TypeScript, that satisfies the constraint that either one of them is provided, but it becomes hard to infer when this type should be used according to the Twitter Docs. And the generators don’t yet support this concept, so it’ll need to be implemented.

I think it’s a great feature for the future, but for now, I think the best solution is to simply make them both optional. This does allow the consumer to call this endpoint with none of them provided, but if so, an error will be thrown from the Twitter API.

[SimonHoiberg]

@iliran11 Yes, totally! I sent you an invite.