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

wip case list #21

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

wip case list #21

wants to merge 1 commit into from

Conversation

alsterjim
Copy link
Collaborator

@alsterjim alsterjim commented Jun 8, 2023

En början på case list, får inte riktigt till typningen, men det är alltid en start

@ballardinimarta
Copy link
Collaborator

Såg att ts bråkade lite, hade rekommenderat att när du skickar in caset till caselist componenten så behöver du inte skicka med hela storyn utan man kanske bara behöver skicka med content i storyn.

const parsedStories = stories.reduce((acc: StoryblokCase[], story: StoryblokStory) => {
		const parsedStory = StoryblokStory.safeParse(story);
		if (!parsedStory.success) {
			throw error(500, 'Could not parse Storyblok story ' + parsedStory.error);
		}
		if (parsedStory.data.content.component !== 'case') {
			throw error(500, 'Wrong content');
		}
		acc.push(parsedStory.data.content);
		return acc;
	}, []);

Så du byter att typen av acc är en lista av StoryblokCase istället för en lista av StoryblokStory

<h1>Selected cases</h1>
<ul>
	{#each bloks as blok}
		<div use:storyblokEditable={blok} class="case">
			<span>No. {index < 10 ? '0' + index : index}</span>
			<span>Case name</span>
			<span>{blok.title}</span>
			<span>Year {blok.date}</span>
		</div>
	{/each}
</ul>

och då kan du byta från blok.content.title/blok.content.date till bara blok.title/blok.date

Annars om du vill få med hela storyn så kan man sätta att

	export let bloks: StoryblokStory[];

men då kommer den inte veta att det är ett case så då tillkommer andra typningsproblem.

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