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

Fix date range params for events #440

Open
wants to merge 2 commits into
base: development
Choose a base branch
from
Open

Fix date range params for events #440

wants to merge 2 commits into from

Commits on Jun 2, 2023

  1. fix(frontend): Fix date range params for events 

    The date range params for when none exist where `moment` objects,
not strings, which are being stringified in the network request.
I'm now formatting the dates appropriately.
    Mark Small committed Jun 2, 2023
    Configuration menu
    Copy the full SHA
    4c59eeb View commit details
    Browse the repository at this point in the history

  2. fix(frontend): Remove unnecessary local state 

    This code was reading global state and assigning it to local state,
that is a big no-no. The selector already causes the component to
re-render when redux is updated.
    Mark Small committed Jun 2, 2023
    Configuration menu
    Copy the full SHA
    5e957c5 View commit details
    Browse the repository at this point in the history