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

Pyodide in form #176

Merged
merged 8 commits into from
May 22, 2024
Merged

Pyodide in form #176

merged 8 commits into from
May 22, 2024

Conversation

thadk
Copy link
Member

@thadk thadk commented May 8, 2024

  • Fix forms so they have default values for easy testing
  • consolidate Github Actions (GHA) rules_engine build process and the rest of the pre-deployment and deployment flow which we mostly inherited from Epic Stack.
  • Change the test of pyodide.test.ts so that it expects(true) for now so we still get notified if it errors but aren't relying on a highly specific fixture which was changing often.
  • Note we haven't fully trialed pyodide on the deployed server quite yet but it might be there.

#187 was also needed to completely resolve the deploy failure due to a new change in our FlyIO host systems.

Also already pushed it to dev branch / heat-stack-staging.fly.io to try it out.

Co-authored-by: Camden Blatchly <[email protected]>
Co-authored-by: plocket <[email protected]>
@thadk thadk force-pushed the pyodide_in_form branch from d4d75e3 to 5a2fd7d Compare May 14, 2024 23:39
Co-authored-by: Camden Blatchly <[email protected]>
Co-authored-by: plocket <[email protected]>
Co-authored-by: Leopardfoot <[email protected]>
@thadk thadk marked this pull request as ready for review May 15, 2024 19:29
@thadk thadk merged commit cc99217 into main May 22, 2024
8 checks passed
@thadk thadk deleted the pyodide_in_form branch May 22, 2024 00:42
AdamFinkle pushed a commit to AdamFinkle/home-energy-analysis-tool that referenced this pull request May 22, 2024
* import Pyodide in single.tsx

Co-authored-by: Camden Blatchly <[email protected]>
Co-authored-by: plocket <[email protected]>

* Add default values to the form

Co-authored-by: Camden Blatchly <[email protected]>
Co-authored-by: plocket <[email protected]>
Co-authored-by: Leopardfoot <[email protected]>

* Put wheel on server by consolidating python + js builds

Co-authored-by: plocket <[email protected]>

* Put wheel on server by consolidating python + js builds (try2)

Co-authored-by: plocket <[email protected]>

* Put wheel on server by consolidating python + js builds (try3)

Co-authored-by: plocket <[email protected]>

* Put wheel on server by consolidating python + js builds (try3)

Co-authored-by: plocket <[email protected]>

* remove old rules-engine tests-build GHA

* Put wheel on server by consolidating python + js builds

Co-authored-by: plocket <[email protected]>

---------

Co-authored-by: Camden Blatchly <[email protected]>
Co-authored-by: plocket <[email protected]>
Co-authored-by: Leopardfoot <[email protected]>
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.

1 participant