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

Release 2024.8.2 #3018

Merged
merged 37 commits into from
Aug 29, 2024
Merged

Release 2024.8.2 #3018

merged 37 commits into from
Aug 29, 2024

Conversation

PeterNerlich
Copy link
Collaborator

@PeterNerlich PeterNerlich commented Aug 23, 2024

Release notes

  • [ #2007 ] Bulk action for canceling translation process
  • [ #2797 ] Fix automatic url update after changing a title of the page on side by side view
  • [ #2801 ] Hide links to images with empty alt attributes
  • [ #2807 ] Fix a bug where machine-translated pages where considered out-of-date
  • [ #2874 ] Retry mechanism for push notifications that were intended to be sent immediately but failed due to an error
  • [ #2969 ] The translation report shows all language labels on the y-axis
  • [ #2982 ] Include links in imprints for link replacement
  • [ #2989 ] Fix bug where adding an event on the last weekday of every month only adds it in months with 5 weeks
  • [ #2998 ] Ignore soft hyphens when detecting links
  • [ #3001 ] Fix link auto update checkbox always checked
  • [ #3005 ] Add go back link for POIS and events

theresantonie and others added 30 commits August 7, 2024 22:52
…ate-on-sbs-form

Fix automatic slug update on sbs form
…gation

Add menu item "contact" to navigation bar
… immediately (#2874)

* Implement overdue filter for push notifications

Settings can be overridden via environment variables, following the guidelines outlined in https://12factor.net/config

Fix pylint and mypi errors

* Add hint for overdue messages

Fix code style issue

* Update firebase_api_client.py

* Add fcm prefix to setting

* Move setup methods to the top of the class

* Retry sending failed push notifications that were intended to be sent immediately.

* Update integreat_cms/core/management/commands/send_push_notifications.py

Co-authored-by: David Venhoff <[email protected]>

* Add release note

* Fix invalid syntax error

---------

Co-authored-by: Dennis Dierkes <[email protected]>
Co-authored-by: David Venhoff <[email protected]>
…_in_link_replace

Include links of imprints into link replacement
Add release note

Initialize function
…considered-outdated

Fix machine translations being considered as not up-to-date
…bels are displayed (#2977)

* Disable auto-skip on y-axis ticks to ensure all labels are displayed

* Add release note about translation chart a11y improvements
…ges-without-alt-tags

Hide links to images with empty alt attributes
…ft-hyphens

Ignore soft hyphens when detecting links
* Correctly specify last month for rrule


Co-authored-by: Charlotte <[email protected]>
…ternal

Keep internal chat messages internal
…o_reset_currently_in_translation

Add bulk action to reset currently in translation
…pdate

fix link auto update checkbox always checked
@JoeyStk JoeyStk added the blocked Blocked by external dependency label Aug 26, 2024
Copy link
Contributor

@JoeyStk JoeyStk left a comment

Choose a reason for hiding this comment

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

The recurrence rule is sadly still broken and needs to be fixed, before we can release a new version.
Blocked by #3022

@JoeyStk
Copy link
Contributor

JoeyStk commented Aug 26, 2024

Apart from the comment above I everything else seems to work as expected :)

@PeterNerlich
Copy link
Collaborator Author

I plan to add a commit on develop directly to address the release notes affected here by #3024 before merging this release.

deen13 and others added 3 commits August 28, 2024 10:11
* Attach region as analytics label to fcm call

* Implement client to get firebase messaging stats

* Remove generate_access_token.py

* Display average delivered notifications in push_notification_form.html

* Fix translation error

* Implement mock_firebase_credentials fixture

* Use djangos built in cache framework

* Use combination of region slug and language slug as anlytics label

* Adjust logic to combined analytics label

* Revert display avg push notifications in news tab

* Implement backup cron job to store firebase delivery statistics

* Remove translation leftover

* Add dependency to pyproject.toml

* Remove outdated test

* Make client a cached property

* Update integreat_cms/firebase_api/firebase_data_client.py

Co-authored-by: JoeyStk <[email protected]>

* Replace average with total

* Remove teardown method that deletes cache as the cache is not used anymore

* Change method call in cron job to correct method name

* Rename language to language_slug

* Simplify code representing the cron job

* Change cronjob to management command

* Drop print statement

* Add firebase statistic model migration

* Add missing docstring

* Update integreat_cms/cms/migrations/0098_firebasestatistic.py

Co-authored-by: JoeyStk <[email protected]>

* Reformat code with ruff

---------

Co-authored-by: JoeyStk <[email protected]>
@PeterNerlich
Copy link
Collaborator Author

This is no longer blocked. For details see the discussion in #3022

@PeterNerlich PeterNerlich removed the blocked Blocked by external dependency label Aug 28, 2024
JoeyStk
JoeyStk previously approved these changes Aug 29, 2024
Copy link
Contributor

@JoeyStk JoeyStk left a comment

Choose a reason for hiding this comment

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

The only issue I had was right at the beginning. I had to use prune_database.sh to fix a JSONDecodeError. But as far as I can tell this was an issue with my local setup - but maybe it's something to keep an eye on :)

@PeterNerlich PeterNerlich merged commit 567a1f2 into main Aug 29, 2024
4 checks passed
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.

10 participants