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

Venus/test1 #4

Merged
merged 2 commits into from
Aug 11, 2023
Merged

Venus/test1 #4

merged 2 commits into from
Aug 11, 2023

Conversation

venusvli
Copy link
Collaborator

Adjusted the coordinates of one of the signs; still just mostly getting used to using git for collaboration again.

@venusvli venusvli added the enhancement New feature or request label Aug 11, 2023
@netlify
Copy link

netlify bot commented Aug 11, 2023

Deploy Preview for scu-virtual-tours ready!

Name Link
🔨 Latest commit e04315e
🔍 Latest deploy log https://app.netlify.com/sites/scu-virtual-tours/deploys/64d59b8cd0838600087b88e1
😎 Deploy Preview https://deploy-preview-4--scu-virtual-tours.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.

"x": -22.96102305361865,
"y": 1.21997231224456693,
"z":-9.258993120824082
"x": -23.5,
Copy link
Collaborator

Choose a reason for hiding this comment

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

I like this position, looks much better and not in the way of the user's path.

version "7.16.7"
resolved "https://registry.npmjs.org/@babel/code-frame/-/code-frame-7.16.7.tgz"
integrity sha512-iAXqUn8IIeBTNd72xsFlgaXHkMBMt6y4HJp1tIaK465CWLT/fG1aqB7ykr95gHHmlBdGbFeWWfyB4NJJ0nmeIg==
version "7.22.10"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I'm not sure how the yarn.lock got changed because it is supposed to only update when there are dependency changes, meaning package.json was changed. I checked out your branch and ran from my side with yarn install, and didn't update the yarn.lock. So, I guess we don't need to worry about it now but keep an eye out if next time your yarn.lock still gets updated despite no dependency changes. We might be able to find a pattern.

Here is the link talking about yarn.lock: yarnpkg/yarn#4379

@venusvli venusvli merged commit e461aa8 into main Aug 11, 2023
4 checks passed
@venusvli venusvli deleted the venus/test1 branch August 11, 2023 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants