Reactions by ReFlar

Kyrne

1.0.0-beta.2

Fix for a reaction not being saved after another user has reacted - Big thanks to @clarkwinkelmann for helping
Added a catch to prevent old reactions from being converted accidentally in edge cases

SqueakyBlum

This extension mostly works well for me. I've noticed that notifications related to reactions seem to be getting escaped though. https://www.dropbox.com/s/opxvtk12fpythbl/Screen%20Shot%202018-05-02%20at%208.19.07%20PM.png?raw=1

Ralkage

SqueakyBlum can you open an issue in our GitHub repo for this extension?

This is the fastest track towards analyzing and fixing any bugs and every bit of detail helps!

Kyrne

SqueakyBlum

Ralkage

No need for this one, it was a super fast fix

1.0.0-beta.4

Fix a notification issue that caused the identifier of reaction to not show

Please allow Bazaar 60 minutes to pull the update

saturngod

I am using latest, 1.0.0-beta.4 but I cannot do react and it's showing following error.

<br />
<b>Catchable fatal error</b>:  Argument 3 passed to Reflar\Reactions\Notification\PostReactedBlueprint::__construct() must be an instance of Reflar\Reactions\Notification\string, string given, called in /var/www/html/vendor/reflar/reactions/src/Listener/SendNotificationWhenPostIsReacted.php on line 76 and defined in <b>/var/www/html/vendor/reflar/reactions/src/Notification/PostReactedBlueprint.php</b> on line <b>41</b><br />

Frink

saturngod beta4? The latest is beta7

Kyrne

Frink he's talking about the extension

saturngod can you further describe the behavior, such as when it happeneds, php version, etc

saturngod

Kyrne PHP 5.6.

I installed it from composer.

datitisev

1.0.0-beta.4

Fix forum CSS rules being applied to elements not added by reflar/reactions

Please allow Bazaar 60 minutes to pull the update

TheMohitMadan

After enabling the extension, I am able to use it from the admin account, but from all other accounts I am getting a permission error. Any idea how to fix this?

jordanjay29

TheMohitMadan Is the React to Posts permission given to anyone besides admins?

TheMohitMadan

jordanjay29 Aah! My bad.
Thank you for your help ?

Kyrne

If you haven't updated to at least 1.0.0-beta.1 and converted your reactions to the new system do so Now.

The next update will remove the ability to convert your reactions to the 1.0.0 system, and therefore they will be lost, and we cannot help you get them back.

010101

Anybody have an issue with this extension where you go to add a reaction, you type in the identifier, the emoji pops up, but it is grayed out and doesn't save? (In the admin)

Kyrne

010101 did you change it to emoji instead of icon?

datitisev

010101 That is the emoji preview when you are using a FontAwesome icon, so you can see what it would look like as an emoji. That's not what would appear in a post's reactions.

010101

Kyrne In my defense, I was half asleep. ? That was it.

Kyrne

Known issue:

High Level:

Reactions causes a long loading time upon clearing the cache. This is known, @datitisev and I have already fixed the issue and it will be released in the beta 8 version of this extension.

Analysis

We depended on the EmojiOne library which caused our dist file to be 688kb (which almost doubles the size of the Flarum payload) We fixed this by no longer depending on EmojiOne, creating an emojimap similar to the emoji extension's beta8 emojimap, and making our own emoji resolver function which reduced the file size to ~68kb (90% less if you are counting) bringing it more in line with most extensions! As a bonus, we added a small algorithm that will try to guess what emoji you are trying to add on the admin interface, even if you don't know the exact name!

010101

Kyrne This is exciting! I love reactions!

zeno

Any chance you'd support multiple reactions to count towards gamification?

« Previous Page Next Page »