Skip to content

Issue 60 change #61

Merged
merged 5 commits into from
Aug 14, 2024
Merged

Issue 60 change #61

merged 5 commits into from
Aug 14, 2024

Conversation

tian223
Copy link
Contributor

@tian223 tian223 commented Aug 7, 2024

Since Time Spent, Certificates, and Badges don't have any links associated with them, I did not link them. I can link the latter 2 to Achievements potentially?

@kim1634
Copy link
Contributor

kim1634 commented Aug 7, 2024

@kim1634
Copy link
Contributor

kim1634 commented Aug 7, 2024

Thanks for the modification! One last thing is, can you exclude the changes for #61 if it is not completed? Other than that, they look good to me.

@ckabuo
Copy link
Contributor

ckabuo commented Aug 14, 2024

Great Work Guys

@kim1634
Copy link
Contributor

kim1634 commented Aug 14, 2024

@tian223 I still see the #61 related code in the admin.js. Can you remove the changes?

@tian223
Copy link
Contributor Author

tian223 commented Aug 14, 2024

@kim1634 For some reason my last commit didn't go through, I think it should be good now?

@kim1634 kim1634 merged commit 31de267 into master Aug 14, 2024
Sign in to join this conversation on GitHub.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants