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

DEV: merge discourse post event settings to calendar #490

Merged
merged 1 commit into from
Dec 6, 2023
Merged

Conversation

lis2
Copy link
Contributor

@lis2 lis2 commented Nov 28, 2023

Before, site settings were divided into discourse calendar and discourse events.

This PR is merging them. Also, translation were moved under discourse_calendar key.

Before, site settings were divided into `discourse calendar` and `discourse events`.

This PR is merging them. Also, translation were moved under `discourse_calendar` key.
@@ -305,148 +305,148 @@ en:
week: "Week"
day: "Day"
list: "List"
discourse_post_event:
Copy link
Contributor

Choose a reason for hiding this comment

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

Sorry to move even more cheese here...but can we also change instances of discourse_post_event to discourse_events? I've never understood why it was called that...

Copy link
Contributor

Choose a reason for hiding this comment

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

If it's too hard to do all in one go that's fine.

@@ -62,33 +62,33 @@ en:
event_expired: "Event expired"
holiday_status:
description: "On holiday"
discourse_post_event:
Copy link
Contributor

Choose a reason for hiding this comment

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

Same comment about discourse_post_event here

@lis2 lis2 merged commit 1c5beb4 into main Dec 6, 2023
4 checks passed
@lis2 lis2 deleted the merge-settings branch December 6, 2023 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants