Igor «InoY» Zviagintsev

Results 27 comments of Igor «InoY» Zviagintsev

Any new updates about screen recorder?

Also I think there is only one place for changes that will add support the new methods: [mobx/src/types/observableset.ts](https://github.com/mobxjs/mobx/blob/main/packages/mobx/src/types/observableset.ts) and it means it is not a big deal but can't imagine...

Also I just realised that there also should be added [Object.groupBy](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/groupBy) and [Map.groupBy](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Map/groupBy)

@mweststrate at the [issue comment](https://github.com/mobxjs/mobx/issues/3850#issuecomment-2021398319) you've mentioned that it should be enough to call an original Set.prototype method so I've copypasted a code from a `Observableset.values()` method. Is it correct?...

Also there is a small change in the `package.json`. This field was added by a Node.js's manager of package managers [Corepack](https://nodejs.org/api/corepack.html). It is because I've ran into a problem at...

@urugator I've just fixed the code of intersetion method according to your comment. I have few questions: 1) Should I keep an `makeIterable` code in the new functions? I mean...

> Afaik we don't have to do anything to support these. I think this methods should trigger `reportObserved()` method at least