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

Enhance codebase with new utilities and config updates #1198

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

wooClO
Copy link

@wooClO wooClO commented Nov 4, 2024

Hello. Some files have been changed and new ones have been created to improve performance. Details under each change

Added React version auto-detection
Added important React component rules
Added basic code quality rules
Added parser options
-optimized apt dependencies installation with cache cleanup
-split file copying for better Docker cache utilization
-used --frozen-lockfile instead of --immutable
-moved application build to separate layer
aded constants.ts file to centralize configuration
central index.ts file for utilities exports
created helpers.ts with reusable functions. Added retry function with exponential backoff. Typed all constants with as const
Added automatic retry functionality. Added transaction replacement handler. Improved error handling with specific error messages
Separated transaction status type
Used interface instead of type for TransactionResult
Removed unused imports to optimize code.
Copy link

vercel bot commented Nov 4, 2024

@wooClO is attempting to deploy a commit to the Coinbase Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

vercel bot commented Nov 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
web-base-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Nov 12, 2024 8:35pm

@cb-heimdall
Copy link
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

@wbnns wbnns added state: backlog This is pending further review community contribution This is a contribution from the community labels Nov 8, 2024
@wbnns
Copy link
Member

wbnns commented Nov 8, 2024

@wooClO

Hi there! Thanks for this PR! Confirming this is in our backlog, pending to be reviewed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
community contribution This is a contribution from the community state: backlog This is pending further review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants