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

LinkInBio template with LaunchDarkly integration #272

Merged
merged 12 commits into from
Oct 22, 2024

Conversation

erinmikailstaples
Copy link
Contributor

Adds a link in bio template with a LaunchDarkly integration.

Copy link
Collaborator

@masenf masenf left a comment

Choose a reason for hiding this comment

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

can you please rebase with the .venv directory removed

@erinmikailstaples
Copy link
Contributor Author

erinmikailstaples commented Oct 18, 2024 via email

@erinmikailstaples
Copy link
Contributor Author

@masenf removed the .venv directory — branch updated.

erinmikailstaples:erin-final-edits

@@ -0,0 +1,5 @@
{
"githubPullRequests.ignoredPullRequestBranches": [
"main"
Copy link
Contributor

Choose a reason for hiding this comment

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

Could we ignore this file?

Copy link
Contributor

Choose a reason for hiding this comment

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

or delete it even?

@elviskahoro
Copy link
Contributor

@erinmikailstaples I'm running the tests! Will message you when they're done!

@erinmikailstaples
Copy link
Contributor Author

hey @elviskahoro anything i can do to help with these tests?

looking at it looks like its failing b/c its not picking up the ldclient import — going to check the requirements file.

remove comment so the launchdakrly-server-sdk is installed.
update version
@elviskahoro elviskahoro merged commit c1fe724 into reflex-dev:main Oct 22, 2024
27 checks passed
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