-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
Add fuel-it extension #16266
base: main
Are you sure you want to change the base?
Add fuel-it extension #16266
Conversation
- fix linting errors - downgrade fuels and ai agent kit version - cleanup - update packages - add loading indicators - revert back to `v0.9.0` - done mvp - add fetch functionality - add `fuel-ai-agent` - fetch and show balances - initial commit
Congratulations on your new Raycast extension! 🚀 You can expect an initial review within five business days. Once the PR is approved and merged, the extension will be available on our Store. |
- Update README.md - Update README.md
- Fix fetch linting errors - Fix \'NodeJS\' has no exported member \'Global\' error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi 👋
Thanks for your contribution 💪
I have now tested your extension, and I have some feedback ready for you:
-
You're missing an extension icon, you can generate one here
-
Could you add at least one screenshot of the extension
I'm looking forward to testing this extension again 🔥
Request a new review when you are ready. Feel free to contact me here or at Slack if you have any questions.
extensions/fuel-it/package.json
Outdated
"commands": [ | ||
{ | ||
"name": "fuel", | ||
"title": "Gfeul", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The title here is a bit weird?
extensions/fuel-it/src/lib/utils.ts
Outdated
interface EnvVars { | ||
OPENAI_API_KEY: string; | ||
FUEL_WALLET_PRIVATE_KEY: string; | ||
} | ||
|
||
export const getEnvVars = async (): Promise<EnvVars> => { | ||
dotenv.config(); | ||
return { | ||
OPENAI_API_KEY: process.env.OPENAI_API_KEY || "", | ||
FUEL_WALLET_PRIVATE_KEY: process.env.FUEL_WALLET_PRIVATE_KEY || "", | ||
}; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not just have it in preferences?
thanks for the suggestions. i will make those changes shortly |
- add metadata - add metadata screenshots - update icon name - make requested changes
- update casing of the title - add metadata - add metadata screenshots - update icon name - make requested changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of switching views, I would just use showToast and change the type from loading to success (and clear the form after successfully sending).
extensions/fuel-it/.env.example
Outdated
OPENAI_API_KEY= | ||
FUEL_WALLET_PRIVATE_KEY= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File should be removed 😊
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Update fuel.tsx - Update fuel.tsx - Show success toast and clear form after success
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it Fuel(-)It or Fuelx
My initial name was |
"commands": [ | ||
{ | ||
"name": "fuel", | ||
"title": "Fuelx", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This name doesn't seem meaningful. What could we change it to?
Description
Screencast
Checklist
npm run build
and tested this distribution build in Raycastassets
folder are used by the extension itselfREADME
are placed outside of themetadata
folder