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

Update index.md for research page with P1 and Monarch social links #157

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

sagehrke
Copy link
Member

Attempting to address #84

I have checked that:

  • all pages' content, images, and styles still appear correctly
  • all links still work
  • redirects (e.g deploy-preview-XX--tislab-website.netlify.app/melzoom) are still working

Copy link

netlify bot commented Jan 10, 2025

Deploy Preview for tislab-website ready!

Name Link
🔨 Latest commit 2c2300d
🔍 Latest deploy log https://app.netlify.com/sites/tislab-website/deploys/6781a912b1a189000877945b
😎 Deploy Preview https://deploy-preview-157--tislab-website.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@sagehrke
Copy link
Member Author

sagehrke commented Jan 10, 2025

@chrisroederucdenver no rush - if you have a moment could you help me make this work? the link for linkedin is not coming up properly and the medium and bluesky links aren't showing up at all.

Links to add:

@sagehrke sagehrke changed the title Update index.md for research page Update index.md for research page with P1 and Monarch social links Jan 10, 2025
@chrisroederucdenver
Copy link
Member

chrisroederucdenver commented Jan 13, 2025

@chrisroederucdenver no rush - if you have a moment could you help me make this work? the link for linkedin is not coming up properly and the medium and bluesky links aren't showing up at all.

  • Phenomics first homepage added and working
  • Monarch Youtube added and working
  • Monarch LinkedIn added and working
  • Monarch Medium aded and working
  • Monarch Bluesky added and working

@sagehrke
On the research page? Committed already?
Screenshot 2025-01-13 at 7 23 16 AM
I don't see much else for Monarch but HomePage and github links.

Is this from a specific recent change, or did you just notice them gone? I don't see any changes to research/index.md since your last commit Nov. 12. If I'm not looking in the right place, please be a bit more specific with URLs or filenames.

@sagehrke
Copy link
Member Author

sagehrke commented Jan 13, 2025

Hey @chrisroederucdenver - no, this isn't committed already. It's for this PR (#157 see the changed file) for updates to the research page. I have not committed it because it's not working and that is what I am hoping to get help with.
I am trying to add Linkedin, Medium, Bluesky, and Youtube links under Monarch but all the socials, except Youtube, are not showing up properly on the deploy preview for this PR - I am stuck. I have updated the previous comment with the links that are supposed to be getting added with this PR and have marked done the two that are working properly. I am hoping for help figuring out how to get Linkedin, Medium, Bluesky to show up before committing.

@sagehrke sagehrke marked this pull request as draft January 13, 2025 16:46
@chrisroederucdenver
Copy link
Member

chrisroederucdenver commented Jan 13, 2025

Looking at _data/links.yaml it looks like only linkedin and youtube are supported, not bluesky or medium. Those can probably be added.

LinkedIn throws me to my feed, not the monarch page. It's doing
https://www.linkedin.com/in/company/the-monarch-initiative/ instead of https://www.linkedin.com/company/the-monarch-initiative/ . No clue what the differnce is to youtube, but our template has that "in" in it.

Youtube looks like it's working.

Again, bluesky and medium need to be added to the _data/links.yaml file.

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