pharmaverse / blog

Blogging on the latest, greatest and most spectacular stuff happening around the pharmaverse
https://pharmaverse.github.io/blog/
Apache License 2.0
19 stars 7 forks source link

Closes #132 #136 #138 #139 Re-vamp of the pharmaverse blog #141

Closed manciniedoardo closed 3 months ago

manciniedoardo commented 3 months ago

Closes #132 Closes #136 Closes #138 Closes #139

Thank you for your Pull Request! We have developed this task checklist to help with the final steps of the process. Completing the below tasks helps to ensure our reviewers can maximize their time on your blog post.

Please check off each taskbox as an acknowledgment that you completed the task or check off that it is not relevant to your Pull Request. This checklist is part of the Github Action workflows and the Pull Request will not be merged into the main branch until you have checked off each task.

manciniedoardo commented 3 months ago

Hi all, I did a bunch of updates in this PR to re-vamp our blog:

bms63 commented 3 months ago

This image is missing image

bms63 commented 3 months ago

Such a nice touch!! image

manciniedoardo commented 3 months ago

This image is missing image

Yeah I'm having some issues getting it to display... thought it maybe was just on my side, any ideas?

bms63 commented 3 months ago

This image is missing image

Yeah I'm having some issues getting it to display... thought it maybe was just on my side, any ideas?

it is missing the contributiors.png file in the post

manciniedoardo commented 3 months ago

This image is missing image

Yeah I'm having some issues getting it to display... thought it maybe was just on my side, any ideas?

it is missing the contributiors.png file in the post

can you see it now? because even though i added it, it's still not rendering for me (same with some of the pharmaverse posts, see image below) image

bms63 commented 3 months ago

The images are rendering for me now. Maybe a restart on your session?

manciniedoardo commented 3 months ago

The images are rendering for me now. Maybe a restart on your session?

I'm happy enough if they render for you :) thanks

sadchla-codes commented 3 months ago

@manciniedoardo this is really Nice!! I think I might have found a few more places where pharmaverse is in propcase, should they also be in all lowercase image

I have also found cases where pharmaverse is all capitalized but this might be a bit tricky since some times the upper case pharmaverse is part of the title for that talk the author referred to and other times it's not. Should the instances where it's the author referring to pharmaverse also be consistent (all lower case)

image
manciniedoardo commented 3 months ago

@manciniedoardo this is really Nice!! I think I might have found a few more places where pharmaverse is in propcase, should they also be in all lowercase image

I have also found cases where pharmaverse is all capitalized but this might be a bit tricky since some times the upper case pharmaverse is part of the title for that talk the author referred to and other times it's not. Should the instances where it's the author referring to pharmaverse also be consistent (all lower case) image

Thanks, looks like I missed some, although some of the ones on there are part of capitalised titles or the first word in a sentence, so I left them alone.

manciniedoardo commented 3 months ago

Thanks, looks like I missed some, although some of the ones on there are part of capitalised titles or the first word in a sentence, so I left them alone.

I have now addressed the others that needed changing.