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

✨ Recent activity - merge 'scanned' and 'attended' achievements #1107

Open
4 tasks
sethdaily opened this issue Oct 18, 2024 · 1 comment
Open
4 tasks

✨ Recent activity - merge 'scanned' and 'attended' achievements #1107

sethdaily opened this issue Oct 18, 2024 · 1 comment

Comments

@sethdaily
Copy link
Member

Cc:

Hi Zach

Pain

There is no difference between 'scanned' and 'attended' achievements - but they clutter up the user profile UI in the app

Suggested Solution

Merge those two types of achievements so I only see 1 feed of activity on a user's profile

Tasks

  • profile UI - investigate merging the 'last seen' and 'recent activity' - see image 1
  • profile UI - there should be only 1 section called 'recent activity'
  • achievements - I think this would mean merging 'scanned' and 'attended'
  • get it checked by Goldie then myself

Screenshots

Image
Figure: Image 1 - Having both of these on here is overcomplicated - it should just be 'recent activity'

Image
Figure: Image 2 - I think it would require merging 'scanned' and 'attended'

Thanks!

@matt-goldman-ssw
Copy link
Contributor

Hi @sethdaily

For my 2c worth, while this is not a technically difficult task, I think it needs some consideration in the broader context of the plans for the app. Keeping them separate gives us flexibility that we lose if we merge them. There's nothing stopping us displaying them all the same way in the UI irrespective of the type defined in the back end, so I would suggest at most just updating how we display them.

But I think we need to understand better how we want this to work and what we want the user experience to be. And whether we have bigger plans around location-based functionality.

Do you want to chat about this more in person when you're back from OS?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants