🐛 potential fix for inserts not always being on newlines

[acidjazz]

🔗 Linked issue

resolves #3463

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[vercel]

@acidjazz is attempting to deploy a commit to the NuxtLabs Team on Vercel.

A member of the Team first needs to authorize it.

[pkg-pr-new]

npm i https://pkg.pr.new/@nuxt/content@3464

commit: 3a94df9

[acidjazz]

npm i https://pkg.pr.new/@nuxt/content@3464

commit: 4569f31

This forced break-return fix has been confirmed to resolve our issue, the two INSERTS are now on their own line and the blog entries are no longer missing

[mfreeman451]

lgtm

[farnabaz]

I'm curious to know how is this happening 🤔 In the template generation we join all lines with \n.

content/src/utils/templates.ts

Lines 96 to 107 in 116fcc9

	export const fullDatabaseRawDumpTemplate = (manifest: Manifest) => ({
	filename: moduleTemplates.fullRawDump,
	getContents: ({ options }: { options: { manifest: Manifest } }) => {
	return Object.entries(options.manifest.dump).map(([_key, value]) => {
	return value.join('\n')
	}).join('\n')
	},
	write: true,
	options: {
	manifest,
	},
	})

Is there any simple reproduction to dig the issue?

[acidjazz]

I'm curious to know how is this happening 🤔 In the template generation we join all lines with \n.

content/src/utils/templates.ts

Lines 96 to 107 in 116fcc9

	export const fullDatabaseRawDumpTemplate = (manifest: Manifest) => ({
	filename: moduleTemplates.fullRawDump,
	getContents: ({ options }: { options: { manifest: Manifest } }) => {
	return Object.entries(options.manifest.dump).map(([_key, value]) => {
	return value.join('\n')
	}).join('\n')
	},
	write: true,
	options: {
	manifest,
	},
	})

Is there any simple reproduction to dig the issue?

I wasn't able to get that going since we have almost 200 blog entries :/ - but i can tell you this is working on our end!

[farnabaz]

@acidjazz I pushed a commit to your PR. Can you confirm that it works as expected with my commit?

[acidjazz]

@acidjazz I pushed a commit to your PR. Can you confirm that it works as expected with my commit?

I will confirm in the next hour or so

[acidjazz]

@acidjazz I pushed a commit to your PR. Can you confirm that it works as expected with my commit?

This is confirmed working on our end