mobx-devtools icon indicating copy to clipboard operation
mobx-devtools copied to clipboard

Looking for maintainers

Open andykog opened this issue 6 years ago • 25 comments
trafficstars

Hey, I don't have much time and interest to support this project lately so if somebody wants to maintain it — you're welcome!

andykog avatar Jun 03 '19 12:06 andykog

Hey @andykog, are you looking for maintainers yet? I'm free to contribute, can we schedule a call to discuss the project?

Thanks

dleitee avatar Aug 20 '19 16:08 dleitee

Sure, please ping me in telegram: @andykog or in twitter: @andy_kogut

On Aug 20, 2019, at 19:52, Daniel Leite de Oliveira [email protected] wrote:

Hey @andykog, are you looking for maintainers yet? I'm free to contribute, can we schedule a call to discuss the project?

Thanks

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

andykog avatar Aug 20 '19 17:08 andykog

I am happy to maintain it @andykog 😄

yuxizhe avatar Oct 12 '19 07:10 yuxizhe

How much leeway would a maintainer have to develop new features?

There are a lot of feature ideas and usability improvements I would like to add to the devtools (and I have enough free time to implement many of them), but I'm weary of having to wade through too much code review and the like. (For libraries, I can understand extreme caution, since mistakes can break production code; but I think it's less important for developer tools, since developers understand the system enough to know how to debug if something goes wrong, and downgrade if necessary.)

I won't intentionally push code that I know introduces issues of course, but it's more a question of whether the implementation choices for new features is able to be done on an "implement first, standardize later" basis, or if instead it's required that we figure out those things ahead of time.

If the latter, an alternate approach I could take is just to create a fork named "mobx-devtools-plus" or something, code all the features there however I like, and then we can backport those features on a case by case basis, as the people here can take a look at what the feature does and how to best standardize around it.

Thoughts? @andykog (and whoever else holds swap in this repo XD)

Venryx avatar Nov 11 '19 17:11 Venryx

@Venryx, maintainer would have unlimited leeway to develop useful features. If developer is not sure whether or not the feature is useful, he/she can discuss it with the community in gh issues. Having code reviews is a very good idea, especially for the first contributions, but nobody will be too strict about stuff like codestyle & test coverage if you’re worrying about that.

andykog avatar Nov 12 '19 08:11 andykog

Okay, sounds good.

For now I went ahead and made this fork: https://github.com/Venryx/mobx-devtools-advanced I'll work there for now, so that I have maximum freedom to iterate quickly.

Once my "initial burst of development" is done, I'll try to form an overview of the changes I made, and present them here. The changes that seem acceptable to add to the main repo, I can then work on porting over.

I think most of the features will probably be worth adding to this repo, however I decided to build it in a (differently named) fork first, since I'm also changing other stuff -- like updating webpack, babel, etc. to the latest versions, and changing the whole project to TypeScript (bit by bit).

Because I'm not sure those more extensive changes would be desired in the main project, using a fork lets me not worry about those other changes while focusing on the new features. Once the intensity of my development evens off, I'll then look to porting the features themselves back to this repo.

If a high enough percentage of the changes are desired in this main repo, I can then discontinue my fork and just work here; otherwise I'll keep it going in parallel and release it as "MobX Developer Tools (Advanced)" or something on the Chrome store.

Venryx avatar Nov 12 '19 08:11 Venryx

I would love to be a co-maintainer. so when you are done ping me for review :)

auvipy avatar Dec 14 '19 08:12 auvipy

Updates:

  • Converted all files (in src folder anyway) to TypeScript.
  • Got basic displaying of component-tree working. (letting you view deep-dependencies of components)
  • Made-so the Changes panel shows the actual old and new value -- even when the values are of a custom type -- for Actions that are logged. (instead of the infuriatingly detail-less "(SomeType)" string)
  • Added UI to Changes tab letting you choose which types of changes you want to show in the Changes list. (greatly helps when trying to debug actions, without reactions filling up the list)
  • Updated a ton of dependencies (webpack, babel, etc.) to their latest versions.
  • Added "Collapse non-MobX" checkbox to components panel. (working, and enabled by default, for now)
  • Changed the default/first-listed font to Roboto. (open source, and also has better alignment, ie. it doesn't have lots of extra space above text like Segoe UI) [latest]
  • Added easy-to-understand error message for when two mobx instances are found in the page. (or rather, when it tries to process a change, where change.object.(the $mobx symbol) is different than collection.mobx.$mobx (the symbol))
  • Added type-data for lots of various constructs.

Screenshot of component tree (very primitive atm):

Screenshot of change filtering by type:

Venryx avatar Dec 15 '19 18:12 Venryx

@Venryx, looks awesome!

andykog avatar Dec 15 '19 22:12 andykog

Updates:

  • Converted all files (in src folder anyway) to TypeScript.
  • Got basic displaying of component-tree working. (letting you view deep-dependencies of components)
  • Made-so the Changes panel shows the actual old and new value -- even when the values are of a custom type -- for Actions that are logged. (instead of the infuriatingly detail-less "(SomeType)" string)
  • Added UI to Changes tab letting you choose which types of changes you want to show in the Changes list. (greatly helps when trying to debug actions, without reactions filling up the list)
  • Updated a ton of dependencies (webpack, babel, etc.) to their latest versions.
  • Added "Collapse non-MobX" checkbox to components panel. (working, and enabled by default, for now)
  • Changed the default/first-listed font to Roboto. (open source, and also has better alignment, ie. it doesn't have lots of extra space above text like Segoe UI) [latest]
  • Added easy-to-understand error message for when two mobx instances are found in the page. (or rather, when it tries to process a change, where change.object.(the $mobx symbol) is different than collection.mobx.$mobx (the symbol))
  • Added type-data for lots of various constructs.

Screenshot of component tree (very primitive atm):

Screenshot of change filtering by type:

when will these changes be released? there is no "Component" tab in current version (0.9.21)

rainmanhhh avatar Apr 28 '20 01:04 rainmanhhh

I became occupied with other work, so didn't finish preparing the changes for a pull-request to this main repo.

You can try out the new Component tab by compiling and installing my forked version locally: https://github.com/Venryx/mobx-devtools-advanced/blob/master/Docs/Hacking.md

Only tested on Chrome atm.

Venryx avatar Apr 28 '20 04:04 Venryx

Hi @andykog :)

I am a huge fan of MobX and I've really missed your dev tools. But right now I am working as a consultant and also I feel so sorrow that I am not able to programming in my job right now. So my proposition is, I can bring to live your application.

My first goal will be fix the missed component and state tab in chrome. Is it okay for you?

Mowinski avatar Nov 20 '22 16:11 Mowinski

Hi @Mowinski, sure, would be nice!

andykog avatar Nov 20 '22 17:11 andykog

@andykog Hey.Thank you so much! I really appreciate your kind words! I used to use Redux for state management, but my current team is using MobX. I really miss the days of having developer tools for state management. Unfortunately, the developer tools for this excellent state management library are no longer maintained. If given the opportunity, I would be more than happy to take on the role of maintaining this project and improving the development experience.

This is a devtool that I recently implemented, which shows my sincere interest in taking on the role of maintainer. Can you give me a chance?

https://github.com/mobxjs/mobx-devtools/assets/160824233/c5caf127-a2a1-4251-a2c1-2a5e96c61ff6

acqyzz avatar Feb 22 '24 06:02 acqyzz

Hi, @acqyzz, looks awesome! Cold you show the code? You can contact me in telegram: @andykog

andykog avatar Feb 22 '24 07:02 andykog

Hey folks, I'm one of the maintainers over at mobx-state-tree and I'm very interested in supporting the broader MobX ecosystem.

I'm focused on MST for the foreseeable future, but if there are small tasks I can assist with, please let me know. My email is in my GitHub profile.

coolsoftwaretyler avatar Mar 11 '24 15:03 coolsoftwaretyler