-
Notifications
You must be signed in to change notification settings - Fork 3
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
"Hide" HTML support from API #894
Comments
|
This was referenced Aug 12, 2024
elsand
pushed a commit
that referenced
this issue
Aug 12, 2024
Planning to only allow HTML for legacy imports #894 Other minor grammar fixes
4 tasks
oskogstad
added a commit
that referenced
this issue
Sep 3, 2024
<!--- Provide a general summary of your changes in the Title above --> ## Description This "hides" HTML support from the API, only allowed with legacy scope. Scope is to be used when importing old stuff from Altinn2/3 to Dialogporten ## Related Issue(s) - #894 ## Verification - [ ] **Your** code builds clean without any errors or warnings - [ ] Manual testing done (required) - [ ] Relevant automated test added (if you find this hard, leave it and we'll help out) ## Documentation - [ ] Documentation is updated (either in `docs`-directory, Altinnpedia or a separate linked PR in [altinn-studio-docs.](https://github.com/Altinn/altinn-studio-docs), if applicable) Co-authored-by: Knut Haug <knut.espen.haug@digdir.no> Co-authored-by: Magnus Sandgren <5285192+MagnusSandgren@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Introduction
This removes the support for HTML in
additionalInfo
and other content propertiesDescription
In order to support rich content, a subset of HTML has been allowed in
additionalInfo
. Frontchannel embeds, which should be used for most content payloads/text bodies, will only support markdown (CommonMark flavour).HTML is intrinsically a markup language that allows for definition of layout, not just rich content. This is in contrast to markdown, which is designed to allow for definition of rich content only. Supporting HTML has challenges related to eg. nesting of tags, which makes it harder to control how rich content is placed within a layout.
Supporting only markdown makes it easier to preserve desired layout, whilst providing fairly rich facilities for text formatting. Note that CommonMark specification will not be supported in full; some features might not be supported or in other ways constrained in order to maintain layout/responsitivy, eg. tables.
For migration purposes however, HTML support will have to be supported. Use of HTML for these scenarios will require an additional oauth scope to be present.
See also #329
Implementation
If there are guidelines on architecture or other implementation choices, they are added here. Different approaches can also be discussed here.
Tasks
The text was updated successfully, but these errors were encountered: