Телеграмм чат группы ts

@Akiyamka привет. Подскажи плиз. Как ты типизируешь или типизировал бы fetch? У нас сейчас вот такая обёртка над ним

type Request = <T>(method: string, url: string, data?: unknown) => Promise<T>;

const request: Request = async (method, url, data) => {
  const h = new Headers();
  h.append('Content-Type', 'application/json');

  const options: RequestInit = {
    method: method.toUpperCase(),
    headers: h,
  };

  if (data) options.body = JSON.stringify(data);

  const res = await fetch(url, options);

  if (res.ok) {
    return res.json();
  }
  throw await res.json();
};

Alex CherryTea in TypeScript — русскоговорящее сообщество

19:53пожаловаться #1

@Akiyamka привет. Подскажи плиз. Как ты типизируешь или типизировал бы fetch? У нас сейчас вот такая обёртка над ним

type Request = <T>(method: string, url: string, data?: unknown) => Promise<T>;

const request: Request = async (method, url, data) => {
  const h = new Headers();
  h.append('Content-Type', 'application/json');

  const options: RequestInit = {
    method: method.toUpperCase(),
    headers: h,
  };

  if (data) options.body = JSON.stringify(data);

  const res = await fetch(url, options);

  if (res.ok) {
    return res.json();
  }
  throw await res.json();
};

Я вроде не спец по TS-у, чтобы советы раздавать)
Ну так на вскидку вижу что не обработали парсинг json-a и код ошибки

John in TypeScript — русскоговорящее сообщество

19:57пожаловаться #2

https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/node-fetch/index.d.ts

@Akiyamka привет. Подскажи плиз. Как ты типизируешь или типизировал бы fetch? У нас сейчас вот такая обёртка над ним

type Request = <T>(method: string, url: string, data?: unknown) => Promise<T>;

const request: Request = async (method, url, data) => {
  const h = new Headers();
  h.append('Content-Type', 'application/json');

  const options: RequestInit = {
    method: method.toUpperCase(),
    headers: h,
  };

  if (data) options.body = JSON.stringify(data);

  const res = await fetch(url, options);

  if (res.ok) {
    return res.json();
  }
  throw await res.json();
};

GitHub

DefinitelyTyped/DefinitelyTyped

The repository for high quality TypeScript type definitions. - DefinitelyTyped/DefinitelyTyped

Denis ø Skiba in TypeScript — русскоговорящее сообщество

19:57пожаловаться #3

всегда в проектах одним из первых смотрю как составлена кастомная request функция)

19:58пожаловаться #4

Denis ø Skiba

всегда в проектах одним из первых смотрю как составлена кастомная request функция)

и что тебе говорит моя request функция?

19:58пожаловаться #5

https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/node-fetch/index.d.ts

John

GitHub

DefinitelyTyped/DefinitelyTyped

The repository for high quality TypeScript type definitions. - DefinitelyTyped/DefinitelyTyped

я, к своему стыду, так и не понял. Можно ли юзать node-fetch на клиенте?

Denis ø Skiba in TypeScript — русскоговорящее сообщество

19:59пожаловаться #6

что сколько людей столько и мнений на такой счет)
я нечего плохого не имел ввиду

Max Syabro in TypeScript — русскоговорящее сообщество

19:59пожаловаться #7

я, к своему стыду, так и не понял. Можно ли юзать node-fetch на клиенте?

так это же вроде полифилл для фетч просто

Max Syabro in TypeScript — русскоговорящее сообщество

19:59пожаловаться #8

не?

19:59пожаловаться #9

хз

Alex CherryTea in TypeScript — русскоговорящее сообщество

19:59пожаловаться #10

я обычно паршру ответы бекенда как-то так:

const clone = response.clone()
try {
const json = await response.json()
} catch (error) {
const responseRaw = response.text();
console.debug(${response.url}: ${responseRaw});
throw new Error(error);
}

Alex CherryTea in TypeScript — русскоговорящее сообщество

20:00пожаловаться #11

и хорошо б проверять statusCode - фетч кидает ошибку только на сетевые ошибки

John in TypeScript — русскоговорящее сообщество

20:01пожаловаться #12

я, к своему стыду, так и не понял. Можно ли юзать node-fetch на клиенте?

это пример, можешь на базе него сделать свой, я не использую fetch, у меня все на rxjs observer обертках