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

First stable release 🎉 #1213

Merged
merged 65 commits into from
Feb 21, 2022
Merged

First stable release 🎉 #1213

merged 65 commits into from
Feb 21, 2022

Conversation

timobrembeck
Copy link
Member

@timobrembeck timobrembeck commented Feb 21, 2022

Short description

First stable release of the new content management system for the Integreat app 🦄 🚀

Changelog

  • [ #1162 ] Allow management role to delete imprint
  • [ #765 ] Add extended view tests
  • [ #765 ] Add tests of form submissions
  • [ #1163 ] Fix error when editor creates new page
  • [ #1165 ] Fix bulk action button for sub pages
  • [ #1173 ] Fix bug where unused location is preselected for new event
  • [ #1166 ] Fix creation of location from event form
  • [ #1172 ] Fix filtering for locations in event list
  • [ #1184 ] Allow user to embed live content from current region
  • [ #1185 ] Fix feedback API
  • [ #1188 ] Fix error in broken link checker
  • [ #1179 ] Disable browser cache of page tree
  • [ #1190 ] Add possibility to set custom region prefix
  • [ #1164 ] Fix possibility to cancel translation process
  • [ #1175 ] Don't show empty tag if the page has subpages
  • [ #1200 ] Fix parent page select input
  • [ #1196 ] Track API requests with Matomo
  • [ #1209 ] Support legacy PDF API
  • [ #1212 ] Only show xliff export option for expert users
  • [ #988 ] Add browser warning when leaving unsaved forms
  • [ #1208 ] Allow editor role to publish events
  • [ #1208 ] Hide feedback and imprint for editor and event manager role

timobrembeck and others added 30 commits February 13, 2022 04:52
Only provide the context data, let Django handle the rendering process automatically
The attribute is called extra_context instead of base_context.
Also, it's more flexible to use get_context_data() in separate context mixins, because this method can be called from the inherited models even if the extra_context attribute is overwritten.
This makes it possible to recognize whether an url is an API or sitemap url by inspecting the resovler match object.
Also, it avoids accidental name conflicts with cms urls.
- Add AccessControlMiddleware which checks the authentication status on every request, independently from the views
- This makes all view-based checks for login, staff status or region permissions obsolete
- Split cms urls in protected urls and public urls with separate namespace
- All api urls and the urls whitelisted in the cms.urls.public module do not require a login
- Add generic list view for simple object lists without customization
- Use this generic list for languages, organizations, offer templates, roles and the language tree
- Provide default ordering for all models
- Use path() instead of url()
- Use simple argument parsing instead of regex
- Consistent use of trailing slashes
- Use dash instead of underscore in url patterns
It should be possible to view forms with the view permission if they are only retrieved and not submitted
- Hide actions tab if user does not have the permission to delete the imprint
- Grant imprint deletion permission to management role
- Use pytest instead of standard unittests
- Add test users for all roles to test data
- Test views for all roles and unauthorized users
- Use parametrization to generate view tests
- Use pytest-xdist to run tests in parallel
- Add tests package to documentation
- Fix the ValueError: '<Page (id: None)>' needs to have a value for field 'id' before this many-to-many relationship can be used.
- Add test case for this bug
Add sub page event listeners for bulk action button
…-form

Fix POI integration in event form and list
- These views are just tested for anonymous users because they cause a redirect for logged-in users
- The uidb64 of the password reset views are not real uuids, just treat them as strings
- Add test to prevent this error in the future
Allow user to embed live content from current region
Enable both views - with trailing slash and without
- Add fallback status if unchecked
- Improve appearance of linkchecker table
- Add links to test data to catch such errors in the future
timobrembeck and others added 27 commits February 18, 2022 18:41
The change to cached properties had an unwanted side effect on the XLIFF import confirmation page:
Since the existing translation was also used in a form instance to validate the imported translation, it got assigned the changed values of the form instance.
This means that the imported diff only showed the imcoming content, not the changes to the previous import.
- Make it possible again to select no parent page for root pages
- Optimize url patterns for page order table
Don't show empty tag if the page has no subpages
- Remove beta tag from next bump version update
- Create non-pre-release on GitHub
- Adapt media library to new color
- Adapt language tabs in forms

Co-authored-by: Timo Ludwig <[email protected]>
Use `AutoField` for primary keys of linkcheck app
Change colouring, remove dummy items & streamline boxes
…bulk-action

Only show xliff export option for expert users
Co-authored-by: Philip Popien <[email protected]>
Co-authored-by: Timo Ludwig <[email protected]>
- grant permission to publish events to editor
- revoke permission to view feedback and imprint for editor and events manager roles
@timobrembeck timobrembeck requested a review from a team as a code owner February 21, 2022 16:03
Copy link
Member

@svenseeberg svenseeberg left a comment

Choose a reason for hiding this comment

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

🚀

@timobrembeck timobrembeck merged commit 36ca625 into main Feb 21, 2022
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.

7 participants