Skip to content

Conversation

melange396
Copy link
Contributor

@melange396 melange396 commented May 7, 2024

Follow up to #963

Open for debate:

  • blurb/title text
  • position in the carousel rotation
  • bg image

@melange396 melange396 requested a review from RoniRos May 7, 2024 21:45
Copy link

netlify bot commented May 7, 2024

Deploy Preview for cmu-delphi-main ready!

Name Link
🔨 Latest commit 0a1ccb3
🔍 Latest deploy log https://app.netlify.com/sites/cmu-delphi-main/deploys/663aa316efa2690008b882ee
😎 Deploy Preview https://deploy-preview-970--cmu-delphi-main.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.

Copy link
Contributor

@carlynvandyke carlynvandyke left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@RoniRos RoniRos merged commit a9390df into dev May 15, 2024
@RoniRos RoniRos deleted the engmgr_carousel branch May 15, 2024 16:46
melange396 pushed a commit that referenced this pull request May 23, 2024
Add 'careers' to carousel with blurb about eng mgr position
@melange396 melange396 mentioned this pull request May 23, 2024
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.

3 participants