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

Sectionview: fixed existing example + added extra examples #6455

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Yinzy00
Copy link
Contributor

@Yinzy00 Yinzy00 commented Sep 24, 2024

Description

I updated the already existing example which wasn't up to date with the current version of the ManifestSectionView.

I also added a typescript example of how to define the manifest + added a lit component example as reference to the linked element in the manifest.

Type of suggestion

  • Typo/grammar fix
  • Updated outdated content
  • New content
  • Updates related to a new version
  • Other

Product & version (if relevant)

V14, v15

@sofietoft
Copy link
Contributor

Thanks for the PR @Yinzy00 ! 🙌

I'll make sure we get this reviewed as soon as possible!

@sofietoft sofietoft added the state/needs-investigation This requires input from HQ or community to proceed label Sep 25, 2024
}
```

**Using TypeScript**
Copy link
Contributor

Choose a reason for hiding this comment

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

@Yinzy00 this section where the manifest file is made with TS, is it implicit that you are using the bundle type?
https://docs.umbraco.com/umbraco-cms/customizing/extending-overview/extension-types/bundle

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm not using bundle i'm using entrypoint.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

You want me to clarify more?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes please 😄

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I’ve added more context about the project setup for the example. However, I’m not sure if this belongs on this page since it seems to relate more to entrypoint extensions. What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category/umbraco-cms state/needs-investigation This requires input from HQ or community to proceed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants