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

Notifications Subscriptions -> Full Implementation #1555

Merged

Conversation

BogoCvetkov
Copy link
Contributor

@BogoCvetkov BogoCvetkov commented Aug 14, 2023

Depends on BE PR -> podkrepi-bg/api#534

The Following Screens have been Implemented

  1. Notification registration forms
  • Non-logged users - General subscribe
    image
    image
    image

NOTE: No one will go to the footer to subscribe. We should add some pop-up that will show up when a visitor lands on the home page. It should contain an input field for the visitor to subsribe his/her email. So probably there should be a design for that one as well

  • Non-logged users - Campaign subscribe

image
image
image

  • Confirm Subscribe Email
    image

  • Subscribe/Unsubscribe confirmation flow
    Loader

  • Subscribe Success / Fail
    image
    image

-Unsubscribe Success/Fail
image
image

  • Logged User Dashboard Notifications Tab
    image
    image
    image
    image

Logged user campaign modal
image
image

@github-actions
Copy link

github-actions bot commented Aug 14, 2023

✅ Tests will run for this PR. Once they succeed it can be merged.

@igoychev igoychev added the run tests Allows running the tests workflows for forked repos label Aug 16, 2023
@github-actions github-actions bot removed the run tests Allows running the tests workflows for forked repos label Aug 16, 2023
Copy link
Contributor

@igoychev igoychev left a comment

Choose a reason for hiding this comment

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

great work! The PR was reviewed in separate PRs in the fork, hence no comments here!

@igoychev igoychev merged commit e7e1208 into podkrepi-bg:master Aug 18, 2023
11 of 12 checks passed
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