Open JoseRFelix opened 9 hours ago
The latest updates on your projects. Learn more about Vercel for Git ↗︎
Name | Status | Preview | Comments | Updated (UTC) |
---|---|---|---|---|
osmosis-frontend | ✅ Ready (Inspect) | Visit Preview | 💬 Add feedback | Oct 4, 2024 4:58am |
The pull request includes modifications to the README.md
file to improve clarity and update user instructions, particularly regarding the transition to a tRPC stack. Changes were made to the "Contributing" section to include a new build step, and the "Testnet" section was expanded with additional instructions. In the code, the packages/server/src/queries/coingecko/index.ts
file was updated to conditionally assign API URLs based on the presence of an environment variable. Additionally, the packages/web/hooks/use-feature-flags.ts
file saw updates to feature flag values and logic.
File Path | Change Summary |
---|---|
README.md |
Updated instructions for transitioning to tRPC, added build step in "Contributing", expanded "Testnet" section, and noted release suspension. |
packages/server/src/queries/coingecko/index.ts |
Modified PRICES_API_URL and DETAILS_API_URL to be conditional based on COINGECKO_API_KEY , introduced COINGECKO_API_URL . |
packages/web/hooks/use-feature-flags.ts |
Updated feature flags in defaultFlags and adjusted logic for returning flags based on isDevModeWithoutClientID . |
use-feature-flags.ts
file, relevant to the main PR's modifications regarding feature flags and build instructions.use-feature-flags.ts
file by adding a new feature flag, aligning with the main PR's focus on development documentation.Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
What is the purpose of the change:
This PR improves the onboarding process by addressing issues related to the initial build when environment variables have not been set up. Specifically, we replaced the hardcoded Osmosis API URL, which requires an API key, with a fallback to the free Coingecko API. While the Coingecko API has rate limits, it is sufficient for initial onboarding and short-term contributions to the code base.
Additionally, this PR updates the default feature flags to display a more up-to-date UI that aligns closely with the current production version. The README has also been updated to streamline the first installation and dev server setup. With these improvements, setting up the code base is now as simple as following the included instructions.
Linear Task
https://linear.app/osmosis/issue/FE-1146/improve-osmosis-fe-onboarding
Brief Changelog