Skip to content
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

Plugin broken #3

Open
Chinney opened this issue Feb 24, 2018 · 8 comments
Open

Plugin broken #3

Chinney opened this issue Feb 24, 2018 · 8 comments

Comments

@Chinney
Copy link

Chinney commented Feb 24, 2018

The plugin is not working for me (OS: Ubuntu 16.04)

I tracked it down to the React.Proptype component which seems to be undefined in avatar-image.jsx. I am currently trying to fix it.

Console Output:

TypeError: Cannot read property 'string' of undefined
    at Object.<anonymous> (avatar-image.jsx:172:16)
    at Object.<anonymous> (avatar-image.jsx:347:1)
    at Module._compile (module.js:569:30)
    at Object.value [as .jsx] (/snap/mailspring/180/usr/share/mailspring/resources/app.asar/src/compile-cache.js:145:21)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (avatar-factory.jsx:1:1)
    at Object.<anonymous> (avatar-factory.jsx:2:1)
    at Module._compile (module.js:569:30)
    at Object.value [as .jsx] (/snap/mailspring/180/usr/share/mailspring/resources/app.asar/src/compile-cache.js:145:21)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (main.es6:2:1)
    at Object.<anonymous> (main.es6:32:2)
    at Module._compile (module.js:569:30)
    at Object.value [as .es6] (/snap/mailspring/180/usr/share/mailspring/resources/app.asar/src/compile-cache.js:145:21)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Package.activate (/src/package.es6:39:22)
    at PackageManager.activatePackage (/src/package-manager.es6:105:9)
    at setTimeout (/src/package-manager.es6:74:14) Object {url: "/home/felix/snap/mailspring/common/packages/mailspring-avatars/lib/avatar-image.jsx", line: 164, column: 50, pluginIds: Array(0)}column: 50line: 164pluginIds: Array(0)url: "/home/felix/snap/mailspring/common/packages/mailspring-avatars/lib/avatar-image.jsx"__proto__: Objectconstructor: function Object()hasOwnProperty: function hasOwnProperty()isPrototypeOf: function isPrototypeOf()propertyIsEnumerable: function propertyIsEnumerable()toLocaleString: function toLocaleString()toString: function toString()valueOf: function valueOf()__defineGetter__: function __defineGetter__()__defineSetter__: function __defineSetter__()__lookupGetter__: function __lookupGetter__()__lookupSetter__: function __lookupSetter__()get __proto__: function __proto__()set __proto__: function __proto__()
module.exports.ErrorLogger.reportError @ /snap/mailspring/180/usr/share/mailspring/resources/app.asar/src/error-logger.js:101
reportError @ app-env.es6:233
window.onerror @ app-env.es6:151
avatar-image.jsx? [sm]:172 Uncaught TypeError: Cannot read property 'string' of undefined
    at Object.<anonymous> (avatar-image.jsx:172:16)
    at Object.<anonymous> (avatar-image.jsx:347:1)
    at Module._compile (module.js:569:30)
    at Object.value [as .jsx] (/snap/mailspring/180/usr/share/mailspring/resources/app.asar/src/compile-cache.js:145:21)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (avatar-factory.jsx:1:1)
    at Object.<anonymous> (avatar-factory.jsx:2:1)
    at Module._compile (module.js:569:30)
    at Object.value [as .jsx] (/snap/mailspring/180/usr/share/mailspring/resources/app.asar/src/compile-cache.js:145:21)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Object.<anonymous> (main.es6:2:1)
    at Object.<anonymous> (main.es6:32:2)
    at Module._compile (module.js:569:30)
    at Object.value [as .es6] (/snap/mailspring/180/usr/share/mailspring/resources/app.asar/src/compile-cache.js:145:21)
    at Module.load (module.js:503:32)
    at tryModuleLoad (module.js:466:12)
    at Function.Module._load (module.js:458:3)
    at Module.require (module.js:513:17)
    at require (internal/module.js:11:18)
    at Package.activate (/src/package.es6:39:22)
    at PackageManager.activatePackage (/src/package-manager.es6:105:9)
    at setTimeout (/src/package-manager.es6:74:14)
(anonymous) @ avatar-image.jsx? [sm]:172
(anonymous) @ avatar-image.jsx? [sm]:347
@Chinney
Copy link
Author

Chinney commented Feb 24, 2018

Also consider this: reactjs/react-rails#825

@Chinney
Copy link
Author

Chinney commented Feb 24, 2018

Seems like this is an deprecation issue. Components like React.PropTypes and React.createClass are no longer part of the main react module. instead they have been moved into seperate modules. I fixed it locally. I installed and used the following packages insted:

react
proto-types
create-react-class

I know this is not a pretty solution but I'm kinda new to react. Maybe someone with more experience have a look into this and use the mailspring react components instead of local ones? Cheers!

@Striffly
Copy link

Fixed on my fork : Striffly@a74fceb

@kriskhaira
Copy link

kriskhaira commented Sep 23, 2018

@Striffly Your fork works for me, although I get this minor layout issue where the unread icon overlaps over the name of the email sender.

screen shot 2018-09-24 at 4 28 16 am

I'm on the latest Mailspring (Version 1.4.2).

@Striffly
Copy link

Striffly commented Oct 3, 2018

@Striffly Your fork works for me, although I get this minor layout issue where the unread icon overlaps over the name of the email sender.

screen shot 2018-09-24 at 4 28 16 am

I'm on the latest Mailspring (Version 1.4.2).

Fixed ! Striffly@876280b

@paulgaumer
Copy link

Hi, thanks a lot @Striffly for adapting this plugin! It doesn't seem to work with Mailspring 1.6.1 though, I don't have any avatars appear in my email list. Does it work for you?

@Striffly
Copy link

Striffly commented Apr 14, 2019

Hi @paulgaumer

Same thing for me, I don't find solution for the moment 😕

@Striffly
Copy link

Striffly commented Oct 10, 2019

Plugin fixed here : https://github.com/Striffly/mailspring-avatars
Due to the inactivity of the author, please go here for future issues.

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

No branches or pull requests

4 participants