-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add a radius to the Recent Activity feed #292
Comments
From the app, we don't have an ability currently to allow the user to expand the radius due to a maximum being imposed by the API of 30 miles https://github.com/scottwainstock/pbm/blob/2106c90f6d8be428830ea63efbd4eb6b506ed7bb/app/controllers/api/v1/user_submissions_controller.rb#L20. We could easily make the radius selectable as it is in the events feed if the API would support such behavior. Personally, I'm not sure how people are using this feature... the 30 mile radius is being applied to the lat/lon of the current map extant. If it would enhance some people's experience to be able to select a different radius value, that seems cool. I don't really use this feature in a way where I find myself wishing this was available personally. |
The users that have requested this live in places like Japan and Finland, where stuff is sparse and far apart! I'm not super into this idea, but they keep requesting it! So yeah, we should start with a pbm issue to... add an endpoint for this value, instead of baking it in. |
We could potentially not show the options for a larger radius if there are greater than X entries in the list for the current map extant. That way the feature would only be available in sparse areas. |
I just looked into this and this doesn't actually need API work! It's true that we have |
For the record: I took one stab at this, then Beth did it better. Branch here. https://github.com/pinballmap/pbm-react/tree/recent-activity Currently merged into master and will be included in 5.2.18 release. |
Really annoying how github doesn't show exact dates for all commits. "two weeks ago" "last month" etc. Anyway, I added the new submission fields on Mon Jan 23, 2023 - which means that by the time our next app release is out, it will be 30 days and so we can potentially use the new fields for this radius. So I'll think about how we can do that, and will comment here with it. |
New user_submission fields:
submission_types: NEW_LMX
NEW_CONDITION
REMOVE_MACHINE
NEW_SCORE
CONFIRM_LOCATION
|
User feedback:
Similar to how you can change the radius to the Events feed.
This makes sense in areas like Japan, but less sense in areas like Southern California. Thoughts?
This is the last open item from the 5.0.0 feedback issue.
The text was updated successfully, but these errors were encountered: