Closed (fixed)
Project:
User Badges
Version:
8.x-1.0-beta1
Component:
Code
Priority:
Critical
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
30 Jan 2017 at 17:51 UTC
Updated:
13 Oct 2017 at 07:13 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
mylies commentedComment #3
mylies commentedComment #4
pifagor commentedThere should be a code like patches. Not tested because they do not have a chance.
Comment #5
pifagor commentedor
Comment #6
proweb.ua commented#5 Now the page is opened without errors
Comment #7
proweb.ua commented#4 here there was a link Add Badge

but if you click on the link Add Badge
Comment #8
mohit_aghera commentedThanks for submitting patch. I'll take a look at it today. Was stuck in some tasks.
Comment #9
mohit_aghera commentedHi @proweb.ua @pifagor
Route to create new badge already exits. I have updated appropriate route there.
Moreover, fixed few more issues found during testing.
Updating patch so others can also review it and discuss it.
Comment #10
Anonymous (not verified) commentedHi,
I'm trying to get this working for my community project drupaldynamics.com - there's been one commit since the beta release which fixes the route name issue in the action links yaml which you've also got in this patch.
I had many issues trying to get the latest dev as it would only give me the beta version but discovered if I added --prefer-source to composer-require it brought down all the branches so I managed eventually to re-roll the patch so it now applies cleanly to the 8.x-1.x branch.
It's breaking in a few other places too so if I manage to fix stuff I'll be posting more ;)
Comment #12
mohit_aghera commentedThanks pifagor, stevepurkiss, proweb.ua
I have pushed your changes and created new release 8.x-1.0-beta2
I still need to do some work in refining UX for backend management of badges, that i will be doing in next release along with bug fixes.
If you have any suggestions on better UX let me know, we can plan it and work accordingly.
Comment #14
pifagor commented