Skip to content
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.

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

Kristina profile added #351

Merged
merged 10 commits into from
Feb 1, 2025
Merged

Conversation

KristinaGagalova
Copy link
Contributor

I have added my profile info and image. Please let me know if there is anything that must be changed!
Thank you

Copy link

netlify bot commented Jan 30, 2025

Deploy Preview for nextflow-staging ready!

Name Link
🔨 Latest commit 3e4ae51
🔍 Latest deploy log https://app.netlify.com/sites/nextflow-staging/deploys/679d806dc33f0d0008a434ee
😎 Deploy Preview https://deploy-preview-351--nextflow-staging.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 51
Accessibility: 95
Best Practices: 83
SEO: 92
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

KristinaGagalova and others added 4 commits January 30, 2025 15:18
1. It should be with ambassadors with name starting with K
2. It should be in the active ambassadors group, not alumni
@mribeirodantas
Copy link
Member

Hi @KristinaGagalova !

I moved your info to the proper location on the page, but there is still one thing we need to fix. Your short bio is too long, pushing the icons beyond the box boundaries as you can see in the picture below.
Screenshot 2025-01-30 at 10 59 06

Deleting a few words should fix it. Check a suggestion below:
Screenshot 2025-01-30 at 10 58 54

Can you please fix that so that I can merge?

@mribeirodantas
Copy link
Member

mribeirodantas commented Feb 1, 2025

It was a tiny bit long still, @KristinaGagalova. I replace " and " with " & " and it's ok now. Are you happy with it this way? Can I merge?

@KristinaGagalova
Copy link
Contributor Author

Yes please, we can merge! Than you for fixing that.

@mribeirodantas mribeirodantas merged commit e313510 into nextflow-io:master Feb 1, 2025
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.

2 participants