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

feat(journal): default limit to today's records #1523

Conversation

jniles
Copy link
Contributor

@jniles jniles commented Apr 19, 2017

This commit extends the hacks done for other registries to the Posting Journal, so that it defaults
to today's records as a search. This should speed up the initial startup of the module considerably
while we develop a better representation of our filters on the client.

Addresses #1436 in a hacky way.


Thank you for contributing!

Before submitting this pull request, please verify that you have:

  • Run your code through ESLint. Check out our styleguide.
  • Run integration tests.
  • Run end-to-end tests.
  • Accurately described the changes your are making in this pull request.

For a more detailed checklist, see the official review checklist that this PR will be evaluated against.

Ensuring that the above checkboxes are completed will help speed the review process and help build a stronger application. Thanks!

@ghost
Copy link

ghost commented Apr 19, 2017

☔ The latest upstream changes (presumably 33b167c) made this pull request unmergeable. Please resolve the merge conflicts.

@jniles jniles force-pushed the hotfix-posting-journal-limit-to-today-default-filter branch 4 times, most recently from 59a0986 to dea71fe Compare April 20, 2017 10:19
@jniles jniles force-pushed the hotfix-posting-journal-limit-to-today-default-filter branch from dea71fe to b2578f2 Compare April 20, 2017 10:33
This commit extends the hacks done for other registries to the Posting Journal, so that it defaults
to today's records as a search.  This should speed up the initial startup of the module considerably
while we develop a better representation of our filters on the client.

Addresses IMA-WorldHealth#1436 in a hacky way.
@jniles jniles force-pushed the hotfix-posting-journal-limit-to-today-default-filter branch from b2578f2 to 97f5359 Compare April 20, 2017 12:22
@jniles jniles merged commit 57528e1 into IMA-WorldHealth:master Apr 20, 2017
@jniles jniles deleted the hotfix-posting-journal-limit-to-today-default-filter branch April 20, 2017 12:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant