Having unlocked a trophy should trigger re-checking achievements

Bug #970273 reported by Rafał Cieślak
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu Accomplishments Daemon
Fix Released
Low
Rafał Cieślak

Bug Description

Having unlocked a trophy should trigger re-checking achievements, because it is possible that some of them have been already completed.

Example case: I am a registered user on LP, and have already filled a bunch of bug reports.
I run the accomplishments system for the very first time. The "first bug reported" trophy is locked, therefore I get only the "registered on LP" trophy - which unlocks the "first bug reported" trophy. However, to get that one, I need to either use the "Check Accomplishments" command from the GUI app, or restart the daemon.
The daemon should check whether I have already completed the "first bug report" trophy automatically (without my inconvenience) as soon as the "registered on LP" accomplishment is validated.
Or, in other words, unlocking any new trophy should immediately trigger re-checking all available trophies.

Related branches

Revision history for this message
Jono Bacon (jonobacon) wrote :

I think this makes sense. It would be useful if the script runner specifically runs the accomplishments that were unclocked to save re-running everything again.

Changed in ubuntu-accomplishments-system:
status: New → Confirmed
importance: Undecided → Low
Jono Bacon (jonobacon)
Changed in ubuntu-accomplishments-system:
milestone: none → 0.1
Revision history for this message
Rafał Cieślak (rafalcieslak256) wrote :

Implementing that may get very tricky. Just re-running everything is very simple, but running a list of accomplishments would require 1) determining the list of just-unlocked accomplishments 2) passing that list to run_scripts_* 3) having scriptrunner select and run only these that are on the list. This fix will probably mess up the API a bit.

Revision history for this message
Jono Bacon (jonobacon) wrote :

Agreed. Let's just re-run the whole set again; I think that makes most sense.

Jono Bacon (jonobacon)
affects: ubuntu-accomplishments-system → ubuntu-accomplishments-daemon
Changed in ubuntu-accomplishments-daemon:
milestone: 0.1 → none
milestone: none → 0.1
Changed in ubuntu-accomplishments-daemon:
assignee: nobody → Rafal Cieślak (rafalcieslak256)
Changed in ubuntu-accomplishments-daemon:
status: Confirmed → In Progress
Jono Bacon (jonobacon)
Changed in ubuntu-accomplishments-daemon:
status: In Progress → Fix Committed
Jono Bacon (jonobacon)
Changed in ubuntu-accomplishments-daemon:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.