Skip to content

Commit 49b335e

Browse files
authored
Merge pull request #20 from mgerni/fix-temple-sync-3
fix temple sync final ly
2 parents af2b9d8 + 61eeb85 commit 49b335e

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

templesync.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ def temple_player_data(username: str):
1212
used_id.add(item['id'])
1313
cleaned_player_data.append(item)
1414

15-
return player_data
15+
return cleaned_player_data
1616

1717

1818
def read_json_file(file_path):
@@ -41,7 +41,9 @@ def import_logs(player_name: str, site_tasks: list):
4141

4242
def check_logs(username: str, site_tasks: list, action: str):
4343
def find_by_id(items, target_id):
44-
return [item for item in items if item['id'] == target_id]
44+
print(type(target_id))
45+
46+
return [item for item in items if int(item['id']) == target_id]
4547
def format_completed_tasks(completed_tasks: set):
4648
# iterate over the completed tasks and create a list of dictionaries
4749
formatted_tasks = []
@@ -52,6 +54,7 @@ def format_completed_tasks(completed_tasks: set):
5254
return formatted_tasks
5355

5456
cleaned_player_data = temple_player_data(username)
57+
print(type(cleaned_player_data))
5558
missing_tasks = list()
5659
completed_tasks = set()
5760
for task in site_tasks:
@@ -79,5 +82,6 @@ def format_completed_tasks(completed_tasks: set):
7982
print(task)
8083
return missing_tasks
8184
else:
82-
print(completed_tasks)
83-
return format_completed_tasks(completed_tasks)
85+
sorted_completed_tasks = sorted(completed_tasks)
86+
print(sorted_completed_tasks)
87+
return format_completed_tasks(sorted_completed_tasks)

0 commit comments

Comments
 (0)