export const getUsers = (params?: any, filters?: any) =>
  Request<Page<User>>(
    setURL(`${API_ROOT}/account/users`),
    setMethod('GET'),
    setParams(params),
    setXFilter(filters)
  ).then(response => response.data);

export const getImages = (params: any = {}, filters: any = {}) =>
  Request<Page<Image>>(
    setURL(`${API_ROOT}/images`),
    setMethod('GET'),
    setParams(params),
    setXFilter(filters)
  ).then(response => response.data);

export const updateAccountSettings = (data: Partial<Linode.AccountSettings>) =>
Request<Linode.AccountSettings>(
  setURL(`${API_ROOT}/account/settings`),
  setMethod('PUT'),
  setData(data, UpdateAccountSettingsSchema)
)
.then(response => response.data);

export const createUser = (data: Partial<User>) =>
  Request<User>(
    setURL(`${API_ROOT}/account/users`),
    setMethod('POST'),
    setData(data, CreateUserSchema),
  )
    .then(response => response.data);

export const updateOAuthClient = (clientId: number, data: Partial<OAuthClientRequest>) =>
  Request<OAuthClient>(
    setURL(`${API_ROOT}/account/oauth-clients/${clientId}`),
    setMethod('PUT'),
    setData(data, updateOAuthClientSchema),
  )
    .then(response => response.data);

export const makePayment = (data: { usd: string; cvv?: string }) => {
  /**
   * in the context of APIv4, CVV is optional - in other words, it's totally
   * valid to submit a payment without a CVV
   *
   * BUT if CVV is included in the payload, APIv4 will send an error that CVV must
   * have 3-4 characters.
   *
   * So for example this payload will result in an error
   *
   * {
   *   usd: 5,
   *   cvv: ''
   * }
   *
   * but this is good
   *
   * {
   *   usd: 5
   * }
   */
  if (!data.cvv) {
    delete data.cvv;
  }

  return Request<Linode.Payment>(
    setURL(`${API_ROOT}/account/payments`),
    setMethod('POST'),
    setData(data, PaymentSchema)
  ).then(response => response.data);
};