Closed yashksaini-coder closed 1 week ago
Thank you for submitting your pull request! 🙌 We'll review it as soon as possible. In the meantime, please ensure that your changes align with our chaotic CONTRIBUTING.md. If there are any specific instructions or feedback regarding your PR, we'll provide them here. Thanks again for your contribution! 😊
Name | Link |
---|---|
Latest commit | 0fb998f31bafcdbc6b5cb94842053b1a4599b9ce |
Latest deploy log | https://app.netlify.com/sites/befiteveryday/deploys/672fbe1c8f50e300084da2a8 |
Deploy Preview | https://deploy-preview-465--befiteveryday.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify site configuration.
@yashksaini-coder I say no need of it, for star gazers you have adding around 2000+ lines
@jeevan10017 I believe Only adding a contributors' page or team is not that great or rare if you ask me, I figured how to generate this amazing stargazers animation and see fit to add it so it can be used to showcase the success of the project. It has been successful as it is accepted in other projects tooo, and their PA quite liked this component.
I request you to take another moment, and please think about this. If possible please give this one level 3 as these 2 PRs are my last, #461 #465
@jeevan10017 If there is no need of this, close this PR. @yashksaini-coder We cant review it without PAs concent
✅ Closes: #454 This pull request introduces a new project template called
stargazers
and includes several updates and additions across multiple files to integrate the Remotion library and configure the project. The most important changes include adding dependencies, configuring ESLint and Prettier, setting up caching and fetching of stargazers, and defining the main components and configurations for rendering a video.Dependencies and Configuration:
package.json
: Added Remotion dependencies (@remotion/bundler
,@remotion/cli
,@remotion/renderer
,remotion
) to the project. [1] [2]stargazers/package.json
: Added project-specific dependencies, scripts for development and build, and configured ESLint and Prettier.stargazers/.eslintrc
: Configured ESLint to extend Remotion's configuration.stargazers/.prettierrc
: Configured Prettier with specific formatting rules.stargazers/.gitignore
: Added common ignore patterns.Codebase Setup:
src/main.jsx
: Refactored imports fromreact-router-dom
to be on a single line.stargazers/src/index.ts
: Registered the Remotion root component.stargazers/remotion.config.ts
: Configured Remotion settings such as video image format and render timeout.Caching and Fetching Data:
stargazers/src/cache.ts
: Implemented caching logic for stargazers data using localStorage.stargazers/src/fetch/fetch-data.ts
: Added functions to fetch stargazers data using either the REST API or GraphQL. [1] [2] [3]Component Implementation:
stargazers/src/Content.tsx
: Created theContent
component to display stargazers information.stargazers/src/Main.tsx
: Defined the main component to calculate progress and renderContent
.stargazers/src/Root.tsx
: Set up the root component with Remotion composition and metadata calculation.Styling:
stargazers/src/gh-styles.css
: Added CSS styles for the project.Summary
This PR finalizes the checks for the remotion application and complete generation of the stargazers timeline animation.
Video (mandatory)
fitflex-454.webm
Checklist:
README.md
if its a new page/tech stackcontributing.md
file before contributingAre you contributing under any Open-source programme?
@GAVINESHWAR @jeevan10017 kindly review this PR, at least lvl 3.