Skip to content

Conversation

Loki-Afro
Copy link
Member

@Loki-Afro Loki-Afro commented Mar 12, 2025

Short Description

Links to Ticket and related Pull-Requests

https://ticketsystem.dbildungscloud.de/browse/BC-9255
hpi-schul-cloud/ckeditor#19

Changes

Data-security

Deployment

New Repos, NPM packages or vendor scripts

Screenshots of UI changes

Checklist before merging

  • QA: In addition to review, the code has been manually tested (if manual testing is possible)
  • PO: Any deviation from requirements was agreed with Product-Owner / ticket author / support-team
  • DEV: Every new component is implemented having accessibility in mind (e.g. aria-label, role property)

Notice: Please keep this Pull-Request as a Draft (or add WIP label), until it is ready to be reviewed

@Loki-Afro Loki-Afro changed the title test BC-9255 make changes in hpi-schul-cloud/ckeditor testable in dev environment Mar 12, 2025
@Loki-Afro Loki-Afro marked this pull request as ready for review March 12, 2025 17:10
"@braintree/sanitize-url": "^7.1.1",
"@ckeditor/ckeditor5-vue": "^5.1.0",
"@hpi-schul-cloud/ckeditor": "^1.2.0",
"@hpi-schul-cloud/ckeditor": "^0.0.1-4c867f8376b47589cb9696054927d8ba2bec1e21",
Copy link
Contributor

Choose a reason for hiding this comment

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

Das soll aber nicht mit gemerget werden.

Comment on lines +8 to +9
# Write the .npmrc file only if the GitHub token is provided
# A token is always required to access GitHub's npm registry
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
# Write the .npmrc file only if the GitHub token is provided
# A token is always required to access GitHub's npm registry
# A token to access GitHub's npm registry is added here conditionally. We use Github's npm reqistry for pre-release versions of our own packages (e.g. @hpi-schul-cloud/ckeditor).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants