Skip to content

add TransDB to projects section - #174

Merged
jonatoni merged 1 commit into
opensourcediversity:mainfrom
Divyamsharma-18:add-transdb-project
Jul 29, 2025
Merged

add TransDB to projects section #174
jonatoni merged 1 commit into
opensourcediversity:mainfrom
Divyamsharma-18:add-transdb-project

Conversation

@Divyamsharma-18

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

Copy link
Copy Markdown
Contributor

📌 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

@Roshanjossey Roshanjossey left a comment

Copy link
Copy Markdown
Member

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. 😓 )

Comment thread 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

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

@Divyamsharma-18

Divyamsharma-18 commented Jun 24, 2025

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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