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

Admin global announcements #132

Merged
merged 2 commits into from
Mar 30, 2018
Merged

Conversation

tstiegler
Copy link
Contributor

For issue #84

I have added a new page for "Front Page Configuration". This page allows you to enter in a markdown announcement that is shown on the front page.

This can be tested by:

  • Logging in as an administrator
  • Clicking the "Front Page Configuration" link under "Admin" in the navigation bar
  • Changing the announcement text
  • Clicking save changes
  • Navigating to the front page (you should observe the defined announcement)

You can remove the announcement by:

  • Clicking the "Front Page Configuration" link under "Admin" in the navigation bar
  • Setting the announcement text to blank
  • Clicking save changes
  • Navigating to the front page (the announcement should no longer be shown)

Copy link
Collaborator

@nationwide13 nationwide13 left a comment

Choose a reason for hiding this comment

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

If possible, hold on merge until @NathanDeckinga can take a look at the migration

@@ -0,0 +1,43 @@
<?php declare(strict_types = 1);
Copy link
Collaborator

Choose a reason for hiding this comment

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

@NathanDeckinga another migration

@TKsM151 TKsM151 merged commit 35cad90 into develop Mar 30, 2018
@TKsM151
Copy link
Contributor

TKsM151 commented Mar 30, 2018

migration

@tstiegler tstiegler deleted the feature-sitewide-announcements-issue-84 branch April 2, 2018 13:26
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.

3 participants