Skip to content

Add AddiDev95 #1246

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.

Sign up for GitHub

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

Open
AdilSaifi04 wants to merge 2 commits into
base: main
Choose a base branch
from
+20 −1
Open

Add AddiDev95 #1246

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7861a36
Add AddiDev95.json with developer profile information
AdilSaifi04 Apr 1, 2026
657ceed
Add 'AddiDev95.json' to ProfilesList.json
AdilSaifi04 Apr 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 18 additions & 0 deletions public/data/AddiDev95.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,18 @@
{
"name": "MD ADIL",
"location": "NOIDA, INDIA",
"bio": "Frontend Devloper Intern ",
Copy link
Copy Markdown

@coderabbitai coderabbitai bot Apr 1, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Correct the bio typo for profile quality.

Line 4 contains a typo: “Devloper” → “Developer”.

🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@public/data/AddiDev95.json` at line 4, Update the "bio" field in
AddiDev95.json to correct the typo: change the value for the "bio" key from
"Frontend Devloper Intern " to "Frontend Developer Intern" (ensure you remove
the extra trailing space as well).

"avatar": "https://github.com/AddiDev95",
"portfolio": "https://github.com/AddiDev95",
"skills": ["React", "GITHUB", "JavaScript", "openSource"],
"social": {
"GitHub": "https://github.com/your-github-username",
"LinkedIn": "https://www.linkedin.com/in/adil-saifi-3ab974262/",
"Instagram": "https://www.instagram.com/adii_worldwide/",
"Email": "saifiadil1600x@gmail.com",
"Discord": "https://discord.com/users/your-discord-id",
"LeetCode": "https://leetcode.com/your-leetcode-username",
Comment on lines +9 to +14
Copy link
Copy Markdown

@coderabbitai coderabbitai bot Apr 1, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix broken social link values before publishing this profile.

Line 9, Line 13, and Line 14 still use template placeholders, and Line 12 should be a URI (mailto:) since href is used directly in src/components/Profile/Profile.jsx.

Suggested patch 
-    "GitHub": "https://github.com/your-github-username",
+    "GitHub": "https://github.com/AddiDev95",
     "LinkedIn": "https://www.linkedin.com/in/adil-saifi-3ab974262/",
     "Instagram": "https://www.instagram.com/adii_worldwide/",
-    "Email": "saifiadil1600x@gmail.com",
-    "Discord": "https://discord.com/users/your-discord-id",
-    "LeetCode": "https://leetcode.com/your-leetcode-username",
+    "Email": "mailto:saifiadil1600x@gmail.com",
     "Reddit": "https://www.reddit.com/user/Itchy-Ad-1484/",
     "Unstop": "https://unstop.com/u/saifiadi98066"
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@public/data/AddiDev95.json` around lines 9 - 14, Update the broken social
link values in the JSON: replace the placeholder GitHub, Discord and LeetCode
values with your actual profile URLs (or remove those keys if you don't have
accounts), and change the Email value to a mailto: URI (e.g., "Email":
"mailto:you@domain.com") because Profile.jsx uses these fields directly in href;
verify the Instagram and LinkedIn remain correct and valid after the edits.

"Reddit": "https://www.reddit.com/user/Itchy-Ad-1484/",
"Unstop": "https://unstop.com/u/saifiadi98066"
}
}
3 changes: 2 additions & 1 deletion src/ProfilesList.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -590,5 +590,6 @@
"Ananya-Hegde2001.json",
"TenathDilusha",
"anmolsah.json",
"hitendras510.json"
"hitendras510.json",
"AddiDev95.json "
Copy link
Copy Markdown

@coderabbitai coderabbitai bot Apr 1, 2026

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

Remove trailing whitespace from filename entry to prevent runtime 404.

Line 594 includes a trailing space ("AddiDev95.json "). Since profile filenames are used directly to build fetch URLs, this will fail to load the new profile.

Suggested patch 
-  "AddiDev95.json "
+  "AddiDev95.json"
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
"AddiDev95.json "
"AddiDev95.json"
🤖 Prompt for AI Agents 
Verify each finding against the current code and only fix it if needed.

In `@src/ProfilesList.json` at line 594, The ProfilesList.json contains a filename
entry with a trailing space ("AddiDev95.json ") which will produce a 404 when
used to build fetch URLs; remove the trailing whitespace so the entry is exactly
"AddiDev95.json" and verify other entries don't contain trailing spaces, or
alternatively ensure the code that loads profiles (where filenames from
ProfilesList.json are used) trims whitespace before constructing URLs (search
for usages of ProfilesList or the string "AddiDev95.json " to locate the entry).

]
Loading