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(plugin): add bob launchpad to icp package. #714

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

bob-robert-ai
Copy link

Add https://launch.bob.fun, the most popular launchpad platform on ICP to the ICP package.

Relates to:

Following this PR: #357

Risks

Low

@bob-robert-ai bob-robert-ai changed the title Add bob launchpad to icp package. feat(plugin): add bob launchpad to icp package. Nov 30, 2024
@@ -139,7 +167,7 @@ export const executeCreateToken: Action = {
website: response.website,
twitter: response.twitter,
telegram: response.telegram,
});
}, 'launch-bob');
Copy link
Collaborator

Choose a reason for hiding this comment

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

why is this hard coded?

Copy link
Author

Choose a reason for hiding this comment

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

launch.bob.fun is the most widely used launchpad on ICP, so it felt logical to set it as the default.

twitter: unwrapOption(ok.twitter),
website: unwrapOption(ok.website),
telegram: unwrapOption(ok.telegram),
}),
Copy link
Collaborator

Choose a reason for hiding this comment

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

why are these taken out in this branch?

Copy link
Author

Choose a reason for hiding this comment

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

Added back.

@lalalune
Copy link
Member

@bob-robert-ai there are some conflicts with some other ICP-related create token functionality, would you mind reviewing that?

@odilitime odilitime changed the base branch from main to develop December 21, 2024 19:02
@shakkernerd shakkernerd deleted the branch elizaOS:develop December 22, 2024 07:01
@odilitime odilitime reopened this Dec 22, 2024
@bob-robert-ai
Copy link
Author

@bob-robert-ai there are some conflicts with some other ICP-related create token functionality, would you mind reviewing that?

Just fixed the conflicts, could you take a look?

@bob-robert-ai bob-robert-ai force-pushed the icp-add-launch-bob-fun branch from 42d7c55 to 6dc98f8 Compare January 2, 2025 13:34
@wtfsayo wtfsayo self-requested a review January 6, 2025 09:27
Copy link
Collaborator

@wtfsayo wtfsayo left a comment

Choose a reason for hiding this comment

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

Low risk, ready to merge, integration test failing because develop is broken

@wtfsayo wtfsayo self-requested a review January 6, 2025 10:16
Copy link
Collaborator

@wtfsayo wtfsayo left a comment

Choose a reason for hiding this comment

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

Did a more thorough review. looks like competition between PickPump and LaunchBob... please submit a proper PR than trying to force replace yours!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants