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

Update fork from upstream #3

Merged
merged 54 commits into from
Jan 16, 2024
Merged

Conversation

jhonatan-lopes
Copy link

Updates this fork from upstream to version 0.10.0.

Cameron Lamb and others added 30 commits June 25, 2021 11:27
* Cleanup RichTextBlockWithFootnotes

* Make sure we have a context to work from and add a comment

* Remove unnecessary step from readme

* Add footnotes as a feature when using RichTextBlockWithFootnotes

* Update changelog
* Make Footnotes unique on page-uuid

* Add migration

* Fix migration and Footnote.DoesNotExist exception

* Footnote.DoesNotExist is no longer needed

We no longer run .get(), instead we store a list of the page's footnotes, we will get a KeyError if there is an invalid footnote reference.
Co-authored-by: Dan Braghis <dan.braghis@torchbox.com>
Only attempt to add footnotes if the context's 'page' is a Page object
The keys in ``self.footnotes`` need to be strings in order for lookups to work, but before the ``Footnote`` is saved to the db, it is of type ``UUID``.

Refs torchbox#23
Cast ``Footnote.uuid`` to string so previews work
Drop Django constraint and let Wagtail dictate it
Add GitHub Action to publish to PyPI on release
They need an alternative template file and javascript file
nickmoreton and others added 24 commits October 14, 2022 16:34
Adds apps file to specifcy auto field
* Drop support for Wagtail <  4.1
* Drop support for Python 3.7
* Update pre-commit hooks
* Add tests
* Add support for Wagtail 5+
these are draft notes and are not up ready yet
* Add Wagtail 5.2 and Python 3.12 in test matrices, update CHANGELOG.md
* Wagtail 5.2 upgrade consideration: Block.get_template now accepts a value argument
* Remove instances of Wagtail 5.0 in the test matrix, Update README.md
---------

Co-authored-by: Katherine Domingo <katherine.domingo@torchbox.com>
@mmmavis mmmavis self-requested a review January 16, 2024 17:58
@jhonatan-lopes jhonatan-lopes merged commit 6be714e into master Jan 16, 2024
1 check passed
@jhonatan-lopes jhonatan-lopes deleted the chore/update-fork-from-upstream branch January 16, 2024 18:01
jhonatan-lopes added a commit that referenced this pull request Jan 16, 2024
* Cleanup RichTextBlockWithFootnotes (#2)

* Cleanup RichTextBlockWithFootnotes

* Make sure we have a context to work from and add a comment

* Remove unnecessary step from readme

* Add footnotes as a feature when using RichTextBlockWithFootnotes

* Update changelog

* Fix replace_footnote_tags by passing 'value' through and correct features

* Footnote make Page and UUID unique together (torchbox#16)

* Make Footnotes unique on page-uuid

* Add migration

* Fix migration and Footnote.DoesNotExist exception

* Footnote.DoesNotExist is no longer needed

We no longer run .get(), instead we store a list of the page's footnotes, we will get a KeyError if there is an invalid footnote reference.

* Bump version to 0.7.0

* Only attempt to add footnotes if the context's 'page' is a Page object

* Update wagtail_footnotes/blocks.py

Co-authored-by: Dan Braghis <dan.braghis@torchbox.com>

* Cast ``Footnote.uuid`` to string so previews work

The keys in ``self.footnotes`` need to be strings in order for lookups to work, but before the ``Footnote`` is saved to the db, it is of type ``UUID``.

Refs torchbox#23

* update wagtail 3.0 code changes

* update package information

* update CHANGELOG

* Bump version and update changelog

* Fix typo

* Modify setup.py so package can be detected by GitHub (torchbox#31)

* Bump to version 0.8.0

* Drop Django constraint and let Wagtail dictate it

* Adds apps file to specifcy auto field

* Add GitHub Action to publish to PyPI on release

* Allow wagtail 4

* Alter footnotes admin css selector

* Adjust footnotes template to match the new style

* Wagtail 2.15,2.16 & 3.0 adjustments

They need an alternative template file and javascript file

* Fixed typo in template name (torchbox#44)

* Revise version to 0.9.0

* Update CHANGELOG.md

* Update CHANGELOG.md

* Updates to support Wagtail 5 (torchbox#49)

* Drop support for Wagtail <  4.1
* Drop support for Python 3.7
* Update pre-commit hooks
* Add tests
* Add support for Wagtail 5+

* Switch to using ruff

* Lint

* Remove unused template/JS

* Tidy up GitHub Actions

* Switch to tox 4

* Test against Wagtail 5.1 (and drop 4.2)

* Tidy up coverage configuration

* Build and use wheel in CI

* Add coverage report in CI

* Switch to using PyPI trusted publishing (torchbox#53)

* Switch to using flit for package build (torchbox#54)

* Drop the docs folder

these are draft notes and are not up ready yet

* Update the CHANGELOG

* Bump version to 0.10.0

* Added basic contributing notes (torchbox#56)

Refs torchbox#55

* Add formal Wagtail 5.2 support (torchbox#63)

* Add Wagtail 5.2 and Python 3.12 in test matrices, update CHANGELOG.md
* Wagtail 5.2 upgrade consideration: Block.get_template now accepts a value argument
* Remove instances of Wagtail 5.0 in the test matrix, Update README.md
---------

Co-authored-by: Katherine Domingo <katherine.domingo@torchbox.com>

* Adapt syntax for Wagtail 4

* Reorder migration

---------

Co-authored-by: Cameron Lamb <cameron.lamb@torchbox.com>
Co-authored-by: Alex Bridge <alex.bridge@torchbox.com>
Co-authored-by: Dan Braghis <dan.braghis@torchbox.com>
Co-authored-by: John-Scott Atlakson <24574+jsma@users.noreply.github.com>
Co-authored-by: Brian Xu <brian.xu@torchbox.com>
Co-authored-by: Kevin <kbhowbrook@gmail.com>
Co-authored-by: Kevin <2128707+kevinhowbrook@users.noreply.github.com>
Co-authored-by: Oliver Parker <oliver@dev.ngo>
Co-authored-by: Nick Moreton <nickmoreton@me.com>
Co-authored-by: nick.moreton <nick.moreton@torchbox.com>
Co-authored-by: zerolab <dan@zerolab.org>
Co-authored-by: Kat <katdom13@gmail.com>
Co-authored-by: Katherine Domingo <katherine.domingo@torchbox.com>
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.

9 participants