Project

General

Profile

Actions

Defect #107

closed

Používání fetch v javascriptu = nefunkčnost v nezanedbatelné části prohlížečů

Added by Zdeněk Burda over 8 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
High
Assignee:
Radek Tomiška
Category:
Frontend
Target version:
Start date:
08/21/2016
Due date:
% Done:

100%

Estimated time:
Affected versions:
Owner:

Description

Na frontendu se používá fetch, ale v nezanedbatelné části mobilních i desktopových prohlížečů je to zatím nepodporované.

http://caniuse.com/#feat=fetch

Safari při přístupu na FE hlásí do konzole "ReferenceError: Can't find variable: fetch", konkrétně končí na volání "return fetch(this.getUrl(path), fetchConfig);" v RestApiService.

```
vodik:src tsunami$ grep -r fetch\( .
./modules/core/services/RestApiService.js: return fetch(this.getUrl(path), fetchConfig);
./modules/core/services/RestApiService.js: return fetch(this.getUrl(path), fetchConfig);
./modules/core/services/RestApiService.js: return fetch(this.getUrl(path), fetchConfig);
./modules/core/services/RestApiService.js: return fetch(this.getUrl(path), fetchConfig);
./modules/core/services/RestApiService.js: return fetch(this.getUrl(path), fetchConfig);
./modules/core/services/RestApiService.js: return fetch(this.getUrl(path), {
./modules/core/services/RestApiService.js: return fetch(this.getUrl(path), {
./services/AttachmentService.js: return fetch(RestApiService.getUrl(this.getApiPath() + `/upload`), {
```

Než se to prohlížeče naučí, nešlo by použít něco jako https://github.com/github/fetch ?


Related issues

Related to IdStory Identity Manager - Task #84: Code reviewClosedOndřej Kopr08/09/2016

Actions
Actions #1

Updated by Radek Tomiška over 8 years ago

  • Assignee changed from Vít Švanda to Zdeněk Burda

Pro fetch se používá js knihovna isomorphic-fetch, která bohužel v rámci #84 vypadla. O víkendu jsem si zkoušel projekt rozjet na windowsích (bez problému) a otestoval aplikaci pod ie edge (se stejným problémem, jak píšeš). Spravil jsem požití isomorphic-fetch (vypadl import), v iečku už to maká. Kdyžtak to prosím zkus otestovat i v safari.

Actions #2

Updated by Radek Tomiška over 8 years ago

  • Status changed from New to Needs feedback
Actions #3

Updated by Radek Tomiška over 8 years ago

  • Related to Task #84: Code review added
Actions #4

Updated by Zdeněk Burda over 8 years ago

  • Status changed from Needs feedback to Resolved

Otestováno v Safari 9.1.2 a funguje to správně.

Actions #5

Updated by Zdeněk Burda over 8 years ago

  • Status changed from Resolved to Closed
  • % Done changed from 0 to 100
Actions #6

Updated by Radek Tomiška over 5 years ago

  • Tracker changed from Task to Defect
  • Category changed from 20 to Frontend
  • Assignee changed from Zdeněk Burda to Radek Tomiška
Actions

Also available in: Atom PDF