rtfeldman/seamless-immutable

Does not work with Javascript Maps

Open

#171 geöffnet am 14. Nov. 2016

Auf GitHub ansehen
 (9 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (5.367 Stars) (221 Forks)batch import
enhancementhelp wanted

Beschreibung

I am using a javacript map to store data I retrieve from an api for a udacity project.

However, the data is not being saved to the immutable object.

here is a branch you can clone and verify:

  1. clone https://github.com/noahehall/udacity-trainschedule/tree/serviceworkers
  2. npm install
  3. npm run start

here is picture of implementation

here is picture of error received from immutable error TypeError: immutableDoesNotWorkWithJavascriptMaps.data.has is not a function(…)

Is there a workaround to get immutable to use Javascript maps? am i doing something wrong (very likely...)? or should I abandon seamless immutable?

Contributor Guide

Tech Stack
javascript
Domain
data
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 days
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
JavaScriptMap objectimmutable data concepts
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
30
Research-Richtung
The issue requests support for JavaScript Map objects in seamless immutable. Currently, the library only handles plain objects and arrays. To implement, examine the immutable wrapper pattern for objects (e.g., overriding get, set, etc.) and adapt it for Map. The user provided a reproduction branch (https://github.com/noahehall/udacity trainschedule/tree/serviceworkers). Review comments for any maintained guidance. The fix would involve creating an ImmutableMap class that mirrors the API but uses Map's internal storage.