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.

Sign up for GitHub

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

Jump to bottom

Allow user selectable areas #518

Merged
merged 8 commits into from
Jun 23, 2022
Merged

Allow user selectable areas #518

merged 8 commits into from
Jun 23, 2022

Conversation

TurtIeSocks
Copy link
Collaborator

  • Let users filter out which areas they want to view stuff in
  • Update getAreaSql to account for userAreas
  • Does NOT override areaRestrictions
  • This PR also, finally filters out which areas a user can see based off of area restrictions
  • Version bump

Addresses #505

@TurtIeSocks
Allow user selectable areas
789c88b 
- Let users filter out which areas they want to view stuff in
- Update getAreaSql to account for userAreas
- Does __NOT__ override areaRestrictions
- This PR also, finally filters out which areas a user can see based off of area restrictions

Addresses #505
@TurtIeSocks
Lint fixes
61a773f
@TurtIeSocks
Alt approach to gql versioning
837060a
@TurtIeSocks
Update scanAreas.js
b7c1151
@TurtIeSocks
Fixes
e67307b 
- More consistent versioning for client and server
- Better GQL logging logic
- Fallbacks in the gym model
- Minor bug fix in the gym model for sorting filters
@TurtIeSocks
Var rename
5c7f1f9 
- Rename to onlyAreas for consistency with other filters
- Fix version injecting...
@TurtIeSocks
Fixes
de64471 
- Revert version back to parsing the package.json...
- Remove checkmarks when using manual areas
- Filter out stale areas
@TurtIeSocks
Version bump
7999bba 
+ move gql logging back behind dev mode to remove unnecessary logs
@TurtIeSocks TurtIeSocks merged commit 4ca2eef into develop Jun 23, 2022
@TurtIeSocks TurtIeSocks deleted the user-selectable-area-filters branch June 23, 2022 02:47
This was referenced Jun 23, 2022
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@TurtIeSocks