Skip to content

Dev/32 - Pigeonite #2102

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

Merged
merged 40 commits into from
Sep 21, 2023
Merged

Dev/32 - Pigeonite #2102

merged 40 commits into from
Sep 21, 2023

Conversation

FrenjaminBanklin
Copy link
Contributor

@FrenjaminBanklin FrenjaminBanklin commented Jul 18, 2023

https://en.wikipedia.org/wiki/Pigeonite

Closes #1869.
Closes #2005.
Closes #2013.
Closes #2081.
Closes #2104.

This release:

  • Adds read-only copies, which can not be modified but allow owners to synchronize changes from the original. The existing drafts_metadata table is used to track read-only status.
  • Changes the ID of every content node in a document that is created as a copy of another document, including references to those IDs in actions.
  • Improves the Assessment Stats dialog in the module dashboard.
    • Filtering and searching is now possible with a single control, which also functions as both a text input and a dropdown selector for easier searching. First and last name are combined for searching purposes, and date ranges are constrained to available results for convenience.
  • Bumps the version of the Node engine from 14 to 18.
    • This required re-locking the canvas, webpack and postcss dependencies (and their subdependencies) at higher versions for compatibility.
  • Bumps the word-wrap dependency version from 1.2.3 to 1.2.4.
  • Bumps the semver dependency version from 5.7.1 to 5.7.2.
  • Bumps the stylelint dependency version from 13.12.0 to 15.10.0 and adjusted associated dependencies and Stylelint rule configurations.

Mauricio and others added 30 commits August 5, 2021 13:29
…ion to remove regex reserved characters. Corrected post-merge unit tests to remove tests that are no longer relevant.
…ore rendering copy/preview buttons in a module's public page. Adds 'normal' and 'read-only' copy options to a module's public page. Replaces 'edit' button with 'synchronize' button in module dashboard for read-only modules. Adds a new dialog and relevant backend logic to automatically synchronize a read-only copied module with its original if any changes have been made.
… are replaced with new UUIDs. References to the old node IDs in button actions etc. are also replaced with the corresponding new IDs.
… 18. Updated Node version requirements in relevant package.json files, fixed new undefined error message in tests.
Bumps [semver](https://github.com/npm/node-semver) from 5.7.1 to 5.7.2.
- [Release notes](https://github.com/npm/node-semver/releases)
- [Changelog](https://github.com/npm/node-semver/blob/v5.7.2/CHANGELOG.md)
- [Commits](npm/node-semver@v5.7.1...v5.7.2)

---
updated-dependencies:
- dependency-name: semver
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [word-wrap](https://github.com/jonschlinkert/word-wrap) from 1.2.3 to 1.2.4.
- [Release notes](https://github.com/jonschlinkert/word-wrap/releases)
- [Commits](jonschlinkert/word-wrap@1.2.3...1.2.4)

---
updated-dependencies:
- dependency-name: word-wrap
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
…ap-1.2.4

Bump word-wrap from 1.2.3 to 1.2.4
…-on-copy

Reset element IDs in copied modules.
…emoved declaration-use-variable dev dependency. Added stylelint-config-standard-scss, stylelint-declaration-strict-value, and autoprefixer dev dependencies. Replaced all instances of 'sh-waqar/declaration-use-variable' with 'scale-unlimited/declaration-strict-value'. Numerous stylelint rule configuration changes to account for existing code not validating against new rules (for now). Fixed sudden descending specificity failures.
…opies

Add optional read-only mode for copied modules and optional update auto-sync for read-only copies.
@FrenjaminBanklin FrenjaminBanklin marked this pull request as ready for review September 15, 2023 15:44
Copy link
Contributor

@mbusch3 mbusch3 left a comment

Choose a reason for hiding this comment

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

Reviewed each of the new and changed files. Nothing jumped out at me. Still need to run tests on all of the pigeonite functionality. I've made new modules with read-only copies, but haven't tested the new assessment data searching and such.

@FrenjaminBanklin FrenjaminBanklin merged commit e8050b0 into master Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants