Skip to content

Conversation

yamageji
Copy link
Contributor

@yamageji yamageji commented Oct 7, 2024

🔗 Linked issue

📚 Description

image

画像URL:https://deploy-preview-369--vuefes-2024.netlify.app/.netlify/images?w=600&h=600&q=80&fit=cover&url=https:%2F%2Fjjdlwtezpdclgxxagxpj.supabase.co%2Fstorage%2Fv1%2Fobject%2Fpublic%2Fcommon_asset%2F0.9303636313761132.png

これでmainにマージすると、画像が出ないという認識合っていますでしょうか?
可能であれば、マージ後の画像URLを教えて頂きたいです...

📝 Note

@yamageji yamageji self-assigned this Oct 7, 2024
Copy link

netlify bot commented Oct 7, 2024

Deploy Preview for vuefes-2024 ready!

Name Link
🔨 Latest commit fa62e6b
🔍 Latest deploy log https://app.netlify.com/sites/vuefes-2024/deploys/6703ad23fc8f7e0008ebed31
😎 Deploy Preview https://deploy-preview-369--vuefes-2024.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.

@yamageji yamageji requested a review from jiyuujin October 7, 2024 06:29
@jiyuujin
Copy link
Collaborator

jiyuujin commented Oct 7, 2024

@yamageji

これでmainにマージすると、画像が出ないという認識合っていますでしょうか?

合っています

可能であれば、マージ後の画像URLを教えて頂きたいです...

issue にも書いている URL です

/.netlify/images?w=10&h=10&q=50&fit=cover&blur=3&url=https:%2F%2Fjjdlwtezpdclgxxagxpj.supabase.co%2Fstorage%2Fv1%2Fobject%2Fpublic%2Fcommon_asset%2F0.9303636313761132.png

@yamageji
Copy link
Contributor Author

yamageji commented Oct 7, 2024

@jiyuujin
ご確認、ありがとうございます!
もう一点だけすみません。/.netlifyの前はhttps://vuefes.jpとなっていましたでしょうか?

https://vuefes.jp/.netlify/.....
  or
https://vuefes.jp/2024/.netlify/....

@jiyuujin
Copy link
Collaborator

jiyuujin commented Oct 7, 2024

@yamageji

/.netlify の前は https://vuefes.jp となっていましたでしょうか?

何も無かったと記憶しています(それは /.netlify から始まるイメージですね

@yamageji
Copy link
Contributor Author

yamageji commented Oct 7, 2024

@yamageji
Copy link
Contributor Author

yamageji commented Oct 7, 2024

@jiyuujin
以下のissueを参考に修正してみました。
nuxt/image#1330 (comment)

これで本番URLでも画像が出るか自信が無いですが、ご確認お願いいたします。

Copy link
Collaborator

@jiyuujin jiyuujin left a comment

Choose a reason for hiding this comment

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

一度、マージしてみましょう(設定自体、悪くない気がします

@jiyuujin jiyuujin merged commit f7f4ec6 into main Oct 7, 2024
8 of 9 checks passed
@jiyuujin jiyuujin deleted the fix/nuxt-image branch October 7, 2024 09:57
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