Hi, I checked latest both stable 1.9.3+2
and test-passed
and realized that custom badge cannot be granted to user if the reason field is not empty.
Steps to reproduce:
create a new simple badge using font-awasome icon, without SQL
go to admin/users, find desirable user
click Edit badges
select the badge created
type a reason (important!) https://www.cuba-platform.com/discuss/t/redirecting-http-to-https/3977/4
click Grant
you’ll get an error, see details below in the picture.
If the reason field is empty, there is no error.
My forum is in a subfolder, probably that’s a reason of the fail:
2 Me gusta
sam
(Sam Saffron)
7 Marzo, 2018 01:50
2
This looks incredibly likely to be the reason, unable to repro on non-subfolder.
4 Me gusta
saurabhp
(Saurabh Patel)
31 Diciembre, 2018 08:10
3
Hi,
I was able to find the issue. I found it out after seeing that we were returning 404 in the first screenshot instead of 400. We were not taking care of relative_root_url while parsing reason of badge.
I have created PR for the same here:
https://github.com/discourse/discourse/pull/6836
7 Me gusta
tgxworld
(Alan Tan)
11 Enero, 2019 07:00
4
This has been merged. Thank you @saurabhp !
4 Me gusta
tgxworld
(Alan Tan)
Cerrado
14 Enero, 2019 10:00
5
This topic was automatically closed after 3 days. New replies are no longer allowed.