Esri/arcgis-rest-js

decorate feature service attachment responses w/ full urls?

Aperta

#371 aperta il 19 ott 2018

Vedi su GitHubGuida contributor
 (4 commenti) (0 reazioni) (0 assegnatari)TypeScript (126 fork)auto 404
good first issue

Metriche repository

Star
 (389 stelle)
Metriche merge PR
 (Merge medio 11g 4h) (5 PR mergiate in 30 g)

Descrizione

@jPurush was going to add this code that we use in the Hub to Hub.js:

https://github.com/Esri/hub.js/blob/2ada2dfe36253e3fa5b75d4993ff28c41198d45f/packages/events/src/get.ts#L45-L60

I wonder if instead of adding it there if rest-js should be decorating the getAttachments() responses w/ the full url similar to the the suggestion for thumbnail urls #323?

As w/ that issue, I'm a bit wary of appending the token. Also it looks like the ?v= parameter is not necessarily supported by the API, but is something we added in the Hub for cache busting. Not sure if we should add that in all cases.

Alternatively, we could provide a getAttachmentUrl(featureServiceUrl: string, attachmentId: number, authentication?: IAuthenticationManager) util fn and let people .map() over the attachment infos to get the URLs themselves?

Guida contributor

Direzione di ricerca
Studia come sono strutturate le risposte degli allegati e come vengono gestiti gli URL delle miniature nel problema #323. Esamina il codice Hub.js collegato nel problema per comprendere la costruzione degli URL e l'uso dei token, quindi valuta le opzioni per aggiungere un helper URL o decorare le risposte.
Tech stack
typescript
Dominio
api
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitTypeScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70