Skip to content

Conversation

@frozenhelium
Copy link
Contributor

@frozenhelium frozenhelium commented Aug 1, 2025

@frozenhelium frozenhelium force-pushed the feature/tutorial-image-block branch from 4946f6c to 3a4c238 Compare August 1, 2025 02:23
Comment on lines 93 to 109
const result = await createTutorialAsset({
data: {
clientId: ulid(),
file,
mimetype: selectedEnum,
tutorial: tutorialId,
},
});

if (
// eslint-disable-next-line no-underscore-dangle
result.data?.createTutorialAsset.__typename === 'TutorialAssetTypeMutationResponseType'
&& result.data.createTutorialAsset.ok
&& result.data.createTutorialAsset.result
) {
onChange(result.data.createTutorialAsset.result.id, name);
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's handle error

Comment on lines 76 to 83
const geoJsonResponse = await fetch(
previewResponse.tutorialAsset.file.url,
);

const rawGeoJson = await geoJsonResponse.json();

// TODO: validate
setGeoJson(rawGeoJson);
Copy link
Contributor

Choose a reason for hiding this comment

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

Let's handle error

Comment on lines +625 to +627
validate: {
objectGeometry: JSON.stringify(feature.geometry, null, 4),
} satisfies ValidatePropertyInputFields,
Copy link
Contributor

Choose a reason for hiding this comment

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

objectGeometry is string?

Copy link
Contributor

Choose a reason for hiding this comment

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

Added FIXME for now.

@barshathakuri barshathakuri merged commit fff5192 into develop Aug 1, 2025
2 checks passed
@barshathakuri barshathakuri deleted the feature/tutorial-image-block branch August 1, 2025 12:24
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.

4 participants