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

feat: notification when followed account creates a new post #1045

Merged
merged 7 commits into from
Jan 13, 2023
Merged

feat: notification when followed account creates a new post #1045

merged 7 commits into from
Jan 13, 2023

Conversation

jeffsikes
Copy link
Contributor

@jeffsikes jeffsikes commented Jan 13, 2023

Fix #538

Added a bell icon to the Account Header.

Noticed that Masto.js 5.4.0 was just released with the necessary changes, but included your suggested // @ts-expect-error and as any for now.

Here are some alternative options for the notify when account posts icon for your review if you want to see what they look like.

image

@stackblitz
Copy link

stackblitz bot commented Jan 13, 2023

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@sxzz sxzz closed this Jan 13, 2023
@sxzz sxzz reopened this Jan 13, 2023
@sxzz sxzz changed the base branch from feat/538-notify-me-when-account-posts to main January 13, 2023 05:18
@netlify
Copy link

netlify bot commented Jan 13, 2023

Deploy Preview for elk-docs canceled.

Name Link
🔨 Latest commit 342ca85
🔍 Latest deploy log https://app.netlify.com/sites/elk-docs/deploys/63c166ccb979c80008c13198

@sxzz sxzz merged commit 9b1ac59 into elk-zone:main Jan 13, 2023
@jeffsikes jeffsikes deleted the feat/538-notify-me-when-account-posts branch January 16, 2023 02:39
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.

Allow follow with notify = true
2 participants