Can not set mail notification when using social login

souramoo / commentoplusplus

Commento with out of the box patches and updates to add useful features and fixes. Also with one-click deploy to Heroku so you can get up and running fast.

MIT License

391 stars 63 forks source link

Can not set mail notification when using social login #16

Closed Ben02 closed 3 years ago

Ben02 commented 3 years ago

The "Notification Setting" link is not appended due to the line below. People using social login are unable to set the mail notification.

https://github.com/souramoo/commentoplusplus/blob/a020fd622f0c710fff737474215f04c0ead04487/frontend/js/commento.js#L381

That file was changes by this commit. I wonder why the notification setting link should not be appended.

souramoo commented 3 years ago

Ah yes, I removed this to make it look a bit less cramped on mobiles. Will add it back in.

souramoo commented 3 years ago

Replaced this with icons to fix mobile formatting and improve interface in 673ea65