Skip to content

Conversation

Divyamsharma-18
Copy link
Contributor

@Divyamsharma-18 Divyamsharma-18 commented Jun 22, 2025

📌 What I did

  • Added TransDB to the Projects section as per issue add TransDB to projects section #173.
  • Included logo sourced from their official GitHub repository.
  • Ensured the layout stays centered by adding empty columns in the row (since it's a single item).

🔗 Issue Reference

Closes #173

🖼️ Preview

Screenshot 2025-06-23 040657

🧾 Note

My code editor auto-formatted the file, which caused the large diff count (+582 −484).
The actual content change is limited to the new TransDB project card addition.
Let me know if you'd prefer a version with minimal formatting changes — happy to update!

@Divyamsharma-18 Divyamsharma-18 changed the title feat: add TransDB to projects list add TransDB to projects section Jun 22, 2025
Copy link
Member

@Roshanjossey Roshanjossey left a comment

Choose a reason for hiding this comment

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

Hi @Divyamsharma-18, thank you for your work on this change.

Yes, it would be best to have the changes with no formatting changes. I have a minor change suggestion about alt test for the project logo (I have no idea why other alt texts are like that. 😓 )

index.html Outdated
encourage diversity and inclusion at those events.</p>
</a>
<a class="col-sm-6 col-md-4 p-4" href="https://transdb.org" target="_blank" rel="noopener">
<img alt="Trans*DB image link" title="Trans*DB" src="img/projects/transdb.png" />
Copy link
Member

Choose a reason for hiding this comment

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

I think a better alt would be TransDB logo

@Divyamsharma-18
Copy link
Contributor Author

Hey @Roshanjossey! I've made the suggested changes:

  • ✅ Updated the alt text to "TransDB logo"
  • ✅ Removed formatting changes

Let me know if there's anything else to update. Danke schön for your kind feedback! 🙌

@Roshanjossey
Copy link
Member

Hi @Divyamsharma-18, I still see only one commit in this pull request. Maybe it wasn't pushed.

@Divyamsharma-18
Copy link
Contributor Author

Divyamsharma-18 commented Jun 24, 2025

Hi @Roshanjossey! I've now pushed the updated commit with all the suggested changes:

✅ Updated the alt text to "TransDB logo"
✅ Removed the formatting changes

(Sorry for the delay — I had accidentally committed to the wrong branch, but it's all fixed now 😊)

Let me know if anything else needs tweaking. Danke schön!

@Divyamsharma-18
Copy link
Contributor Author

Hey @Roshanjossey! Just wanted to follow up on the earlier comment above regarding the PR and the updates. Let me know if anything else is needed from my side — happy to refine things further! 😊

@Divyamsharma-18
Copy link
Contributor Author

Hey @Roshanjossey!
Just wanted to gently follow up on the PR and comment above. I’ve completed the changes and everything’s updated, would love your feedback when you get a chance!

No rush at all, just wanted to make sure it didn’t get lost in the queue 😊

@Roshanjossey
Copy link
Member

Hi @Divyamsharma-18, thank you for following up. I still see formatting changes in the PR.

+585 −482

I think it's the first commit that's causing this. second commit looks correct.

Also, I'm waiting for a go ahead from a maintainer in TransDB team

@Divyamsharma-18
Copy link
Contributor Author

Hey @Roshanjossey,
Thank you for the reply! I cleaned up the formatting, it’s now just a single commit with the intended changes only. Let me know if anything else needs tweaking 🙌
And of course, happy to wait on the go-ahead from TransDB maintainers as well!

@jonatoni
Copy link
Member

thanks for the work @Divyamsharma-18 and for your support @Roshanjossey 👏

@jonatoni jonatoni merged commit 4d16769 into opensourcediversity:main Jul 29, 2025
1 check passed
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.

add TransDB to projects section
3 participants