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

React input editor #5

Merged
merged 24 commits into from
Nov 27, 2019
Merged

React input editor #5

merged 24 commits into from
Nov 27, 2019

Conversation

reyery
Copy link
Member

@reyery reyery commented Nov 22, 2019 •
edited
Loading

This PR adds editing and saving of schedules to the input editor.

Changes

Fixes

  • Hour Header starting from 0 instead of 1

(build link: https://we.tl/t-6gC0nQuuR0)

@reyery
Fix modal button still loading
c1423bf
@reyery
Update style for schedule editor error text
19aae50
@reyery
Add tooltips to column headers
19b507f
@reyery
Update schedule editor logic for selected buildings
28509fc
@reyery
Add diff function for schedules table
2fe3a2c
@reyery
Fix discard changes message popup too high
dbf8edf
@reyery
Refactor input editor to share table buttons
a14de47
@reyery
Update data of table instead of replacing when data changes
68507de
@reyery
Rewrite updateChanges into a function
e4f33f9
@reyery
Add basic input validation for year table
5704325
@reyery
Adjust column width when selecting new buildings and changing tab
8479e2d
@reyery
Add schedule editor redux actions
9aa8955
@reyery
Add parameter to fetchBuildingSchedule to fetch all buildings
a4fba31
@reyery
Update save and discard changes to include schedules
f8638e7
@reyery
Fix hour header to start from 1 instead of 0
a14b4ce
@reyery
Fix table redrawing before data is fetched
1124d7c
@reyery
Revert adding fetchBuildingSchedule fetch all parameter
93c4b6b
@reyery
Add debouncing to fetching building schedules
a399914
@reyery
Fix change summary showing wrong hour
c211020
@reyery
Enable multi selection while holding ctrl
50552ba
@reyery reyery mentioned this pull request Nov 22, 2019
Merged
@reyery reyery merged commit 026b266 into master Nov 27, 2019
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.

feature request: show GFA where there is information about the area and volume
1 participant
@reyery