Skip to content

Commit

Permalink
UX: clarifies discourse_post_event_enabled needs calendar_enabled
Browse files Browse the repository at this point in the history
  • Loading branch information
jjaffeux committed Apr 12, 2020
1 parent f1f6fe9 commit c195a28
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion config/locales/server.en.yml
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
en:
site_settings:
calendar_enabled: "Enable the discourse-calendar plugin. This will add support for a [calendar][/calendar] tag in the first post of a topic."
discourse_post_event_enabled: "[experimental] Enables to attach an event to a post."
discourse_post_event_enabled: "[experimental] Enables to attach an event to a post. Note: also needs `calendar enabled` to be enabled."
displayed_invitees_limit: "Limit the numbers of invitees displayed on an event."
discourse_post_event_allowed_on_groups: "Groups that are allowed to create events."
holiday_calendar_topic_id: "Topic ID of staffs holiday / absence calendar."
Expand Down

2 comments on commit c195a28

@discoursereviewbot
Copy link

Choose a reason for hiding this comment

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

Robin Ward posted:

Should we do this in a validator?

@discoursereviewbot
Copy link

Choose a reason for hiding this comment

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

Joffrey JAFFEUX posted:

The thing is many method of plugin instance do their magic on "plugin.enabled?" at runtime with it being computed from a specified setting (here calendar_enabled).

Please sign in to comment.