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: Create choose SDK view #89

Merged
merged 3 commits into from
Mar 28, 2024

Conversation

k3llymariee
Copy link
Contributor

@k3llymariee k3llymariee commented Mar 28, 2024

Adds the second step to the quickstart flow to select an SDK. SDKs are sorted by most popular as determined by the following queries:

Which I then dumped into this google sheet to format for our purposes. It's a little manual but these don't change that often. We can also add whatever else we need to display the SDK instructions.

choose-an-sdk.mp4

Copy link

This pull request has been linked to Shortcut Story #238063: create choose SDK view in setup wizard.

l.Styles.TitleBar = lipgloss.NewStyle()
l.SetShowPagination(true)
l.SetShowStatusBar(false)
l.SetFilteringEnabled(false) // TODO: try to get filtering working
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Simply flipping this on to true doesn't work as expected - probably due to the fact that the container model is still delegating to this nested model.

I found this example of a simple list working with filtering, and got it to work on its own, but when I tried implementing those same changes in our delegated list model it still didn't work 😢

Copy link
Contributor

@sunnyguduru sunnyguduru left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for researching and making the spreadsheet

@k3llymariee k3llymariee merged commit 7518423 into main Mar 28, 2024
2 checks passed
@k3llymariee k3llymariee deleted the kelly/sc-238063/create-choose-sdk-view branch March 28, 2024 23:43
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.

3 participants