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

Fd/merge upstream branch 2024 08 28 t1504 0500 #4

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

fidiego
Copy link
Contributor

@fidiego fidiego commented Aug 28, 2024

fidiego and others added 9 commits August 28, 2024 15:08
…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
* theme example: dark card updates

* app: use display name, if available
* 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.
this required a refactor of the root layout and modifications to the way
we handle `Metadata`.

- [ ] figure out how to use `searchParams` in the `generateMetadata`
- [ ] ensure the metadata is generated well regardless of what method we
      use to get the installerId.
Copy link

vercel bot commented Aug 28, 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 28, 2024 8:12pm

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