Fix: Remove unnecessary button and link props from Apply page timeline#662
Fix: Remove unnecessary button and link props from Apply page timeline#662Aarya-Chaudhari wants to merge 3 commits intoAOSSIE-Org:mainfrom
Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Hello @Zahnentferner |
Addressed Issues:
Fixes #645
Problem
On the /apply page, the vertical timeline displays the steps involved in the GSOC application process such as:
Join us on Discord
Start Contributing
Write a Draft Application
Discuss with Mentors
Submit Application
In src/app/apply/page.jsx, each TimelineElement was receiving button and link props. However, these props were not used for navigation or rendering any interactive elements in the UI.
Keeping unused props can be misleading for contributors, as it suggests that the timeline elements are interactive when they are actually meant to be static informational steps.
Solution
Removed the unused button and link props from all TimelineElement usages in:
src/app/apply/page.jsx
Now the component only receives the required props:
Result
The timeline UI and behavior remain unchanged.
The codebase is cleaner and easier to understand.
Removes unnecessary props that could confuse contributors.
Testing
Ran the development server using npm run dev.
Verified the /apply page renders correctly.
Confirmed the timeline steps display exactly as before with no visual changes.
Files Modified
src/app/apply/page.jsx
Screenshots/Recordings:
Code Before:

Code After Changes:

Additional Notes:
AI Usage Disclosure:
We encourage contributors to use AI tools responsibly when creating Pull Requests. While AI can be a valuable aid, it is essential to ensure that your contributions meet the task requirements, build successfully, include relevant tests, and pass all linters. Submissions that do not meet these standards may be closed without warning to maintain the quality and integrity of the project. Please take the time to understand the changes you are proposing and their impact. AI slop is strongly discouraged and may lead to banning and blocking. Do not spam our repos with AI slop.
Check one of the checkboxes below:
I have used the following AI models and tools: TODO
Checklist