Skip to content

Conversation

@joseconsador
Copy link
Contributor

Sideloads the users instead of caching all users on page load.

Note

This PR removes the Priority column from all grids. The reason for this is because to enable sideloading - we must use /search/incremental which does not return the priority field.

/cc @miogalang @jwswj @mmolina @iandjx

References

Risks

  • Medium - Users may not be loaded or an unexpected response is returned by the incremental endpoint

@miogalang
Copy link
Contributor

👍

1 similar comment
@jwswj
Copy link
Contributor

jwswj commented Sep 8, 2015

👍

joseconsador added a commit that referenced this pull request Sep 9, 2015
@joseconsador joseconsador merged commit fcb3298 into master Sep 9, 2015
@joseconsador joseconsador deleted the jose/MI-277-user-caching branch September 9, 2015 05:25
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

Successfully merging this pull request may close these issues.

4 participants