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

Refactor: Cleanup and Organize Project Structure #282

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

Conversation

berat
Copy link

@berat berat commented Oct 14, 2024

Hello,

I've made some updates to our project to help keep things neat and tidy. Here's a breakdown of what I've done:

Removed unused folder

  • I noticed that the pages folder wasn't being used, so I went ahead and removed it. This should help reduce clutter and make our project directory a bit cleaner.

Organized Imports

  • Grouped similar imports together to make our code easier to read and manage. This should help us quickly find what we need when working on different parts of the project.
  • Added an index.ts file to frequently used components. This makes it super easy to import these components without having to remember their exact paths.

Typo

  • I also noticed an important typo in the "edge-rate-limiting.mdx" blog post and fixed it.

I started with a simple change to reduce complexity and create a more organized structure. I plan to contribute to my next PR in small pieces, aiming to be cleaner and reusable.

Copy link

vercel bot commented Oct 14, 2024

@berat is attempting to deploy a commit to the Upstash Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Collaborator

@ademilter ademilter left a comment

Choose a reason for hiding this comment

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

great thanks

Copy link
Collaborator

Choose a reason for hiding this comment

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

why did you delete this file?

Copy link
Author

Choose a reason for hiding this comment

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

I see that it is an unnecessary code. We don't seem to use it anywhere. Could I be wrong?

Copy link
Author

Choose a reason for hiding this comment

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

Screenshot 2024-11-14 at 14 58 03

You are using the package on npm. So that's why it looks like we do not need it.

Copy link

vercel bot commented Oct 30, 2024

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

Name Status Preview Updated (UTC)
website ✅ Ready (Inspect) Visit Preview Dec 5, 2024 3:27pm

…ster

# Conflicts:
#	.prettierrc.json
#	src/app/about/page.tsx
#	src/app/blog/[slug]/page.tsx
#	src/app/blog/all/page.tsx
#	src/app/blog/author/[author]/page.tsx
#	src/app/blog/page.tsx
#	src/app/blog/tag/[tag]/page.tsx
#	src/app/brand/page.tsx
#	src/app/careers/[slug]/page.tsx
#	src/app/careers/page.tsx
#	src/app/customers/[slug]/page.tsx
#	src/app/customers/page.tsx
#	src/app/examples/[slug]/page.tsx
#	src/app/examples/page.tsx
#	src/app/fast/page.tsx
#	src/app/layout.tsx
#	src/app/pricing/layout.tsx
#	src/app/pricing/qstash/page.tsx
#	src/app/pricing/redis/page.tsx
#	src/app/pricing/vector/page.tsx
#	src/components/career/header.tsx
#	src/components/example/client.tsx
#	src/components/example/comp.tsx
#	src/components/example/filter.tsx
#	src/components/home/community/comp.tsx
#	src/components/home/community/index.tsx
#	src/components/home/fast/index.tsx
#	src/components/home/fast/statistic.tsx
#	src/components/home/investor/index.tsx
#	src/components/home/open-source/index.tsx
#	src/components/home/price/index.tsx
#	src/components/home/price/qstash.tsx
#	src/components/home/price/redis.tsx
#	src/components/home/product/index.tsx
#	src/components/home/product/qstash.tsx
#	src/components/home/product/redis.tsx
#	src/components/home/product/vector.tsx
#	src/lib/segment/provider.tsx
@berat
Copy link
Author

berat commented Nov 14, 2024

Updated branch and fixed conflicts

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.

2 participants