Open andrewdang17 opened 2 years ago
See #381
Hey @andrewdang17 and @adamkovalsky! Sorry it took so long for anyone to get to this.
I'm going through and tagging/responding/closing out issues in MobX-State-Tree. Looks like the answer is sets are not supported, but I think we'd be happy to see a PR to that effect.
Much like #381, I'm going to mark this as help/PR accepted. Hopefully someone jumps in if they're looking for a good issue to grab. If not, I imagine a maintainer will try and get this done in the coming months/year.
If either of you is still interested, I'd be happy to get you set up with contributing. If not, no worries.
Hey @coolsoftwaretyler I would be happy to contribute to this issue, Can you tell me what should i do about this issue?
Hey @a-hassanzadeh-h - thanks for your interest! I haven't thought too much about it, but here's what I would do to get started:
Let me know what you think! As a heads up, I'm going to be a little busy in the coming weeks. Sorry for any delay in responses.
Hey @a-hassanzadeh-h have you picked this??
Hey, @chakrihacker I was busy in the last few days, But I am working on issue #1939 right now, I can work on this issue in the next week, If anybody else wants take this issue right now, it is okay, they welcome
I will give it a shot then
Question
I'd like to use a Set for one of the model properties but it seems like converting the set to an array and using
types.array(types.number)
would be my only option. If I try to use it anyways I get an empty array and this typescript error