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

Add Announcements #603

Open
wants to merge 24 commits into
base: live
Choose a base branch
from
Open

Add Announcements #603

wants to merge 24 commits into from

Conversation

davidbeig
Copy link
Member

@davidbeig davidbeig commented Apr 2, 2024

🎩 What? Why?

Please describe your pull request.

This PR allows the admins to create announcements for the community of Goteo.

Announcements are shown in the home page, and can be added to any other layout and it's controllers, to show some content to the users.

There are still some developments missing:

  • Admin to create Announcements
  • Add slider to handle multiple announcements
  • Add translations
  • Add LocalStorage support to not be visible everytime a user enters.
  • Add announcements to Projects landing
  • Add announcements to Channels

Testing

Describe the best way to test or validate your PR.

Run the migrations

bin/console migrate up

Create an entry for the announcement in the database. It should have a call to action and a type such as: project, general, donation

📷 Screenshots

Announcement for donations to the platform in tablet view and in mobile view.
Announcement of type donation in tablet view

Mobile view for a donation announcement

Announcement to call to fill fiscal details
imatge

imatge

♥️ Thank you!

@davidbeig davidbeig requested a review from subiabre April 2, 2024 13:57
@davidbeig davidbeig self-assigned this Apr 2, 2024
@davidbeig davidbeig changed the title Feature/announcement Add Announcements Apr 2, 2024
subiabre
subiabre previously approved these changes Jun 5, 2024
@davidbeig davidbeig marked this pull request as ready for review September 16, 2024 13:04
@davidbeig davidbeig mentioned this pull request Sep 20, 2024
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.

2 participants