I'm going to also add a feature request to the GitHub repo for this project to add the option to have different sizes for the post displayed achievement icon and the achievement modal that pops up. This will help with the sizing issue so it'll behave like platforms such as Reddit where the post achievement icons are small and the achievement received modal has the bigger image.

Reddit award modal:

Reddit award modal

The post awards (count not required):

Reddit post awards

Edit: Done! malago86/flarum-achievements2

trying to setup a new achievement, but when i click save i get the oops something went wrong error:

console:
Failed to load resource: the server responded with a status of 500 (Internal Server Error) api/achievements:1

malago Here you go:

[2021-04-03 10:11:59] flarum.ERROR: ParseError: syntax error, unexpected 'Arr' (T_STRING), expecting ')' in /web/htdocs/www.zerodx.it/home/forum/vendor/malago/flarum-achievements/src/Api/Controllers/CreateAchievementController.php:66

      enricodx I made this change days ago but for some reason it did not make it to the GitHub repository... I'm still learning how to use all these tools 🙄

      Version 0.1.2

      • Fixed missing comma
      • Added some tweaks to the achievements' image size

      Please update:
      composer update malago/flarum-achievements

          malago Great! An important feature to add, in my opinion, is the ability to edit and remove previous achievement.

              enricodx I am not sure I understand what you mean.

              You can add/remove achievements manually if you want, although it will only work for achievements that you set as Manual (the others will be added/removed when they are recalculated).

              You can also set a minimum and maximum threshold for some achievements. The way I use this on my forum is something like this, for number of posts:

              • Beginner: from 0 to 49 posts
              • Intermediate: from 50 to 99 posts
              • Advanced: more than 100 posts

              The achievements are removed when they go over the threshold and the next achievement will be given instead (with the notification). Is this what you mean?

                  malago I mean simply remove an achievement at all. For example, I created a Test achievement and now I can't edit or remove it

                      enricodx You can't? You can click on the achievement icon when you created it to edit and delete it. If you did not set an icon, just click above the name of the achievement in the list. It will load the modal again and you can modify it or delete it.

                          malago

                          when i click save i get an error (new achievements...the first)

                          i can't see an ID, maybe is this the issue?

                              malago

                              alert: {type: "error", content: Array(1), controls: false}
options: {method: "GET", url: "http://flarum.localhost/flarum/public/api/achievements", params: {…}, background: true, config: ƒ, …}
response: null
responseText: "<br />↵<b>Warning</b>:  Trying to access array offset on value of type null in <b>D:\xampp\htdocs\flarum\vendor\malago\flarum-achievements\src\Api\Serializers\AchievementSerializer.php</b> on line <b>47</b><br />↵<br />↵<b>Warning</b>:  Trying to access array offset on value of type null in <b>D:\xampp\htdocs\flarum\vendor\malago\flarum-achievements\src\Api\Serializers\AchievementSerializer.php</b> on line <b>47</b><br />↵<br />↵<b>Fatal error</b>:  Uncaught Laminas\HttpHandlerRunner\Exception\EmitterException: Output has been emitted previously; cannot emit response in D:\xampp\htdocs\flarum\vendor\laminas\laminas-httphandlerrunner\src\Exception\EmitterException.php:24↵Stack trace:↵#0 D:\xampp\htdocs\flarum\vendor\laminas\laminas-httphandlerrunner\src\Emitter\SapiEmitterTrait.php(40): Laminas\HttpHandlerRunner\Exception\EmitterException::forOutputSent()↵#1 D:\xampp\htdocs\flarum\vendor\laminas\laminas-httphandlerrunner\src\Emitter\SapiEmitter.php(27): Laminas\HttpHandlerRunner\Emitter\SapiEmitter-&gt;assertNoPreviousOutput()↵#2 D:\xampp\htdocs\flarum\vendor\laminas\laminas-httphandlerrunner\src\RequestHandlerRunner.php(98): Laminas\HttpHandlerRunner\Emitter\SapiEmitter-&gt;emit(Object(Flarum\Api\JsonApiResponse))↵#3 D:\xampp\htdocs\flarum\vendor\flarum\core\src\Http\Server.php(44): Laminas\HttpHandlerRunner\RequestHandlerRunner-&gt;run()↵#4 D:\xampp\htdocs\flarum\public\index.php(26): Flarum\Http\Server-&gt;listen()↵#5 {main}↵  thrown in <b>D:\xampp\htdocs\flarum\vendor\laminas\laminas-httphandlerrunner\src\Exception\EmitterException.php</b> on line <b>24</b><br />↵"
status: 500
xhr: XMLHttpRequest {readyState: 4, timeout: 0, withCredentials: false, abort: ƒ, onreadystatechange: ƒ, …}
__proto__: Object

                                  Justoverclock That's a new one! Can you check your database to see if the flarum_achievements_user table has a column named new?

                                      Justoverclock So the table exists, but it is empty? If the structure matches the one I sent, it is correct...

                                      I can't reproduce this on my end so I will have to investigate more... It might be that the original migration was wrong and that messed up the tables? Maybe redoing the tables helps?

                                      CREATE TABLE `flarum_achievements` (
  `id` int(10) UNSIGNED NOT NULL,
  `name` varchar(200) COLLATE utf8mb4_unicode_ci NOT NULL,
  `description` text COLLATE utf8mb4_unicode_ci NOT NULL,
  `computation` varchar(100) COLLATE utf8mb4_unicode_ci NOT NULL,
  `points` int(11) NOT NULL,
  `image` varchar(255) COLLATE utf8mb4_unicode_ci NOT NULL,
  `rectangle` varchar(100) COLLATE utf8mb4_unicode_ci NOT NULL,
  `active` tinyint(1) NOT NULL DEFAULT '1'
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;


ALTER TABLE `flarum_achievements`
  ADD PRIMARY KEY (`id`);


ALTER TABLE `flarum_achievements`
  MODIFY `id` int(10) UNSIGNED NOT NULL AUTO_INCREMENT;

CREATE TABLE `flarum_achievement_user` (
  `id` int(10) UNSIGNED NOT NULL,
  `user_id` int(10) UNSIGNED NOT NULL,
  `achievement_id` int(10) UNSIGNED NOT NULL,
  `created_at` timestamp NULL DEFAULT NULL,
  `updated_at` timestamp NULL DEFAULT NULL,
  `new` tinyint(1) NOT NULL DEFAULT '0'
) ENGINE=InnoDB DEFAULT CHARSET=utf8mb4 COLLATE=utf8mb4_unicode_ci;

--
-- Índices para tablas volcadas
--

--
-- Indices de la tabla `flarum_achievement_user`
--
ALTER TABLE `flarum_achievement_user`
  ADD PRIMARY KEY (`id`),
  ADD KEY `flarum_achievement_user_user_id_achievement_id_index` (`user_id`,`achievement_id`),
  ADD KEY `flarum_achievement_user_achievement_id_foreign` (`achievement_id`);

ALTER TABLE `flarum_achievement_user`
  MODIFY `id` int(10) UNSIGNED NOT NULL AUTO_INCREMENT;
  
ALTER TABLE `flarum_achievement_user`
  ADD CONSTRAINT `flarum_achievement_user_achievement_id_foreign` FOREIGN KEY (`achievement_id`) REFERENCES `flarum_achievements` (`id`) ON DELETE CASCADE,
  ADD CONSTRAINT `flarum_achievement_user_user_id_foreign` FOREIGN KEY (`user_id`) REFERENCES `flarum_users` (`id`) ON DELETE CASCADE;
COMMIT;

                                          malago ah, I just checked your commit history; curious to why you didn’t create a migration to update/make modifications to the table(s) instead?

                                            Version 0.1.3

                                            • Fixed a bug when posting new discussions. This is a temporary fix until I find a better way to do it, but it works 🙂
                                            • Fixed a bug when creating the first achievement

                                            Justoverclock Please check out this version and let me know if your error is fixed!

                                            I am getting close to be confident that the extension is stable. I really appreciate your reports and comments!