Skip to content

Remove deprecated usage of React.PropTypes #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sowiecki
Copy link

@sowiecki sowiecki commented Jul 3, 2017

@gtfiorentino
Copy link

@Nase00 - thanks for the PR. This project isn't really actively maintained due to the popularity of the Redux DevTools browser extension (e.g. for Chrome: https://chrome.google.com/webstore/detail/redux-devtools/lmhkpmbekcpmknklioeibfkpmmfibljd?hl=en), which provides the same functionality and much more.

I'm curious to hear about your use case and whether there's a reason you either can't use the extension, or if switching to the extension is too much of a burden. This should only be a problem if you're otherwise updating package versions, though, unless I'm missing something.

@sowiecki
Copy link
Author

Ah, I see, fair enough. I've preferred to stick with the non-extension version so that I can use monitors that don't seem to be available in the extension. Specifically filter-actions and my own creation, upload-download-monitor. I've also got a bit of custom code to save positional, size, and monitor ordering in local storage, which has been handy keeping everyone on my dev team happy.

Looking at the docs, it doesn't seem possible to use custom monitors with the extension, correct?

@gtfiorentino
Copy link

I don't believe there's a way to hook into the extension for custom monitors, though I honestly haven't looked very hard.

We can look into why the build is failing (think it's a dependency issue) and try to get this merged, though I can't promise a quick turnaround. Do you have a fork that your team is working from?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants