Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Finishing v1 of learning analytics summary #522

Closed
4 tasks
wcjord opened this issue Jul 24, 2024 · 6 comments
Closed
4 tasks

Finishing v1 of learning analytics summary #522

wcjord opened this issue Jul 24, 2024 · 6 comments
Assignees
Labels
enhancement New feature or request

Comments

@wcjord
Copy link

wcjord commented Jul 24, 2024

Feature Description

We have this feature partially complete but need to do a few more things before pushing our v1.

  • For each message, immediately update local totals. We should save a list of recently sent messages and a list of recently created construct uses. The display of total points, level, messages, and construct uses should pull from analytics room + local recent lists. When the app decides to send those updates based on that optimizing logic, empty the recent lists.
  • Move this work from room details to branch from main. Room details has a bigger set of changes and I'd like to get the analytics summary out the door first.
  • Hide link to My Analytics
  • Use weights for constructUseTypeEnum.pointValue for tallying total points for user's level.

Note - we'll allow clicking of the items for accessing message, vocab, error, and level details views in a later push.

Rationale

To incentivize ze chatting.

Mockup

Screenshot 2024-07-24 at 2 17 55 PM

Additional Context

No response

@wcjord wcjord added the enhancement New feature or request label Jul 24, 2024
@wcjord
Copy link
Author

wcjord commented Jul 24, 2024

@ggurdin Let me know if you encounter any technical complications or want to discuss implementation.

@wcjord
Copy link
Author

wcjord commented Jul 26, 2024

@ggurdin We're going to take that total number of messages out of the summary analytics. You can just hide that. We'll have a third number on that line at some point, likely 4th and 5th, but just 2 for now.

@ggurdin ggurdin linked a pull request Jul 31, 2024 that will close this issue
@wcjord
Copy link
Author

wcjord commented Aug 22, 2024

@ggurdin This is all done yeah? Merged into our main? I'd like to roll this out well before story game.

@ggurdin
Copy link
Collaborator

ggurdin commented Aug 22, 2024

It's close to do, yes. I still have some lingering questions about small details with storing constructs during IGC/IT (#582). Should that part of it also be included in this push?

@wcjord
Copy link
Author

wcjord commented Aug 22, 2024 via email

@wcjord
Copy link
Author

wcjord commented Sep 3, 2024

I believe this is done!

@wcjord wcjord closed this as completed Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants