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

Add build options to UI #92

Merged
merged 29 commits into from
Nov 27, 2023
Merged

Add build options to UI #92

merged 29 commits into from
Nov 27, 2023

Conversation

MichaelPesce
Copy link
Collaborator

  • Update design for input page and toolbar
  • Add build options drop down to incorporate any exported options
  • Add rebuild button

relies on #1152

@MichaelPesce
started adding new design for input page
cffc9a9
@MichaelPesce
updated tabs style, added build/re-build button; need to clean this u…
949e858 
…p a bit
@MichaelPesce
added unbuild flowsheet api endpoint, hooked it up to cancel button
1a156b6
@MichaelPesce
moved analysis type dropdown to inside config input; made top section…
a69d1d1 
… just build options
@MichaelPesce
fix typo
3fca01e
@MichaelPesce
default to show build options being false
2fe14de
@MichaelPesce
flowsheet list table update; only have flowsheets for wastewater reco…
7e0d019 
…very sof ar
@MichaelPesce
updated e2e tests
ae4a084
@MichaelPesce
update component test
0abacf0
@MichaelPesce
added check for build options, drop down that uses build options if f…
3ae0fd1 
…ound
@MichaelPesce
moved build options into separate component; added API route and func…
fe8f08d 
…tion for selecting an option; added connection for options and build function
@MichaelPesce
updated build process to check if flowsheet is built on frontend, to …
0a09ae7 
…allow for rebuilding ob backend
@MichaelPesce
updated get_diagram function to check for export provided images
bc4800c
@MichaelPesce
try except around fetch image name
a0e8743
@MichaelPesce
add options to flowsheet list items; add check for options when loadi…
3268c64 
…ng flowsheet - if options are found, dont build flowsheet by default
@MichaelPesce
use my watertap fork for app tests
1d3b2a6
@MichaelPesce
use my watertap fork for app tests
76c52d5
@MichaelPesce
fix workflow
1e225e5
@MichaelPesce
fix workflow
c588c9d
@MichaelPesce
options -> build_options
8953655
@MichaelPesce
Merge branch 'main' of https://github.com/MichaelPesce/watertap-ui in…
1467017 
…to some-cleanup
@MichaelPesce
add additional screenshot to e2e
01b24ba
@MichaelPesce MichaelPesce self-assigned this Sep 27, 2023
@MichaelPesce
Copy link
Collaborator Author

addresses issue #85. also overlaps with PR #90, which can probably be closed

@ksbeattie ksbeattie added the Priority:Normal Normal Priority Issue or PR label Sep 28, 2023
@MichaelPesce MichaelPesce marked this pull request as ready for review October 26, 2023 20:49
@MichaelPesce MichaelPesce merged commit abd1033 into watertap-org:main Nov 27, 2023
4 checks passed
@MichaelPesce MichaelPesce mentioned this pull request Nov 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@MichaelPesce MichaelPesce

Labels
Priority:Normal Normal Priority Issue or PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MichaelPesce @ksbeattie