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.

Sign up for GitHub

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

Jump to bottom

Release 2023.2.1 #2079

Merged
merged 26 commits into from
Feb 21, 2023
Merged

Release 2023.2.1 #2079

merged 26 commits into from
Feb 21, 2023

Conversation

ulliholtgrave
Copy link
Member

@ulliholtgrave ulliholtgrave commented Feb 18, 2023 •
edited

Changelog

  • [ #2056 ] Change restriction of event duration to 28 days
  • [ #2032 ] Remove gap between sidebar boxes in location form
  • [ #1989 ] Improve load time of locations endpoint
  • [ #1578 ] Add option to hide files in global media library
  • [ #1752 ] Show chapter for internal link suggestions
  • [ #1965 ] Add ical rrule for recurring events

timobrembeck and others added 17 commits January 30, 2023 13:59
@timobrembeck
Indicate that opening hours are now shown in the apps
6211882
@timobrembeck
Fix formatting error in slug error message
4003c94
@timobrembeck
Merge pull request #2069 from digitalfabrik/bugfix/slug-error-transla…
4b5f97e 
…tion

Fix formatting error in slug error message
@MizukiTemma @timobrembeck
Change max event duration to 28 Days
1d98b7b 
Co-authored-by: Timo Ludwig <timo.ludwig@tuerantuer.org>
@MizukiTemma
Merge pull request #2059 from digitalfabrik/feature/max_event_duratio…
03562c6 
…n_28_days

Change max. event duration to 28 Days
@timobrembeck
Merge pull request #2034 from digitalfabrik/enhancement/poi-form-open…
31e42ed 
…ing-hours-note

Indicate that opening hours are now shown in the apps
@david-venhoff
Show chapter for internal link suggestions
9900791 
This commit changes the internal link suggestions to show the full
path to an object instead of just its title.
This does not affect the general search feature, because there it is
not ambiguos which object is which after performing the search.
@JoeyStk
Remove gap between sidebar boxes (for pois)
2b2ba3a
@JoeyStk
Merge pull request #2043 from digitalfabrik/bugfix/remove_gap_between…
7876259 
…_collapsed_boxes

Remove gap between sidebar boxes (for POIs)
@ztefanie @timobrembeck
Add debugging section to documentation
c5e0e32 
Co-authored-by: Timo Ludwig <timo.ludwig@tuerantuer.org>
@timobrembeck
Add optional test data for POIs
cd7612d
@ztefanie @timobrembeck
Improve load time of location endpoint
70630ff 
- Use default public translation instead of default translation for POIs
- Prefetch POI categories including their translations
- Prefetch organization

Co-authored-by: Timo Ludwig <timo.ludwig@tuerantuer.org>
@ulliholtgrave
Add shortcut texts
8b85817
@ulliholtgrave
Merge pull request #2016 from digitalfabrik/feature/show_chapter_for_…
6f8819d 
…link_suggestions

Show chapter for internal link suggestions
@ulliholtgrave
Merge pull request #2078 from digitalfabrik/feature/add_shortcut_texts
e4421f1 
Add shortcut texts to editor gui
@ulliholtgrave
Merge pull request #2057 from digitalfabrik/feature/improve-load-time…
35ab8e8 
…-of-location-endpoint

Improve load time of location endpoint
@JoeyStk
Add option to hide fields in global media library
7300c9f
@ulliholtgrave ulliholtgrave marked this pull request as ready for review February 18, 2023 16:11
@ulliholtgrave ulliholtgrave requested a review from a team as a code owner February 18, 2023 16:11
JoeyStk and others added 3 commits February 18, 2023 17:11
svenseeberg
svenseeberg previously approved these changes Feb 19, 2023
View reviewed changes
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 previously approved these changes Feb 19, 2023
View reviewed changes
Copy link
Member

@timobrembeck timobrembeck left a comment

Choose a reason for hiding this comment

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

Should we include #2082 in the release as well?

@ulliholtgrave
Copy link
Member Author

I guess it's worth to wait for Sven's fix.@svenseeberg should I also provide a review?

@timobrembeck timobrembeck dismissed stale reviews from svenseeberg and themself via fa69ef0 February 20, 2023 13:29
@ztefanie
Merge pull request #2029 from digitalfabrik/feature/combine-recurring…
2bfdda5 
…-events

Combine recurring events and add rrule
@svenseeberg
Copy link
Member

I guess it's worth to wait for Sven's fix.@svenseeberg should I also provide a review?

I'll include Timos last comment in the next hours. If there's more, we should probably not wait?

@ztefanie
Copy link
Member

If this can be easily added to the release would be nice, as we can than start working on it in the frontend: #2029

@ulliholtgrave
Copy link
Member Author

If this can be easily added to the release would be nice, as we can than start working on it in the frontend: #2029

@SteffiStuffel Your changes are already part of the release and i adjusted the PR description :)

@ulliholtgrave
Copy link
Member Author

I guess it's worth to wait for Sven's fix.@svenseeberg should I also provide a review?

I'll include Timos last comment in the next hours. If there's more, we should probably not wait?

I agree, I propose that we release it this afternoon and if we are able to fix the remaining issues with the mutex we can include it. Otherwise, we'll bring that fix in a separate release.

svenseeberg and others added 2 commits February 21, 2023 14:25
@svenseeberg @timobrembeck
Add mutex to page tree move function, fix #1518
e43e493 
* Use django-db-mutex module to implement a mutex that allows only
  one page tree move operation at a time.

Co-authored-by: Timo Ludwig <timo.ludwig@tuerantuer.org>
@svenseeberg
Merge pull request #2082 from digitalfabrik/bugfix/tree-beard-mutex
30a8e9a 
Add mutex to tree move function, fix #1518
timobrembeck
timobrembeck approved these changes Feb 21, 2023
View reviewed changes
Copy link
Member

@timobrembeck timobrembeck left a comment

Choose a reason for hiding this comment

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

🚀

@ulliholtgrave ulliholtgrave merged commit b98ea12 into main Feb 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@timobrembeck timobrembeck timobrembeck approved these changes

@svenseeberg svenseeberg svenseeberg left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@ulliholtgrave @svenseeberg @ztefanie @timobrembeck @MizukiTemma @david-venhoff @JoeyStk