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: merge upstream 2024 08 20 t1646 0500 #2

Merged
merged 14 commits into from
Aug 20, 2024

Conversation

fidiego
Copy link
Contributor

@fidiego fidiego commented Aug 20, 2024

fidiego and others added 14 commits July 11, 2024 11:59
…l_markdown (#46)

* fix: update install inputs

* feat: interpolate install.install_inputs.values into the post_install_markdown

* feat: hijack enter for stepper navigation

the stepper is wrapped in one big form element so pressing enter in any
of the steps would submit the form. this is less than intutitive
behavior since users are likely to interpret the sections of the stepper
as distinct forms.

as a result, we opt to hijack/intercept enters on the form and use it to
navigate the stepper instead of submitting the form. now, the only way
to submit the form is via the (re)provision button.

* feat: interpolate install.id into the post install markdown
With the recent changes to render input values in the form, it looks
like the installer could no longer handle installs with no inputs. This
change should fix that.
* theme example: dark card updates

* app: use display name, if available
* feat: mov cloud configs after the app configs

* feat: add card drop shadows back and in theme
* theme: update default button shadow

* logo: ensure largest logos render
* feat: ignore .env files

* feat: delegation aware installer

only works for ios. is backwards compatible.
the last step is special in the sense that its state depends on the
state of the Stepper/Steps AND on the state of the Install. in this
commit, we introduce a method for dynamically determining the last
Step's activeClassName to achieve the desired state-based border
color.
Copy link

vercel bot commented Aug 20, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
demo-installer ✅ Ready (Inspect) Visit Preview Aug 20, 2024 9:47pm

@fidiego fidiego changed the title Fd/merge upstream 2024 08 20 t1646 0500 feat: merge upstream 2024 08 20 t1646 0500 Aug 20, 2024
@fidiego fidiego merged commit 06d3fc8 into main Aug 20, 2024
4 checks passed
@fidiego fidiego deleted the fd/merge-upstream-2024-08-20T1646-0500 branch August 20, 2024 22:51
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.

2 participants