-
Notifications
You must be signed in to change notification settings - Fork 33
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 in the app and website top bars #787
Conversation
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/labelflow/labelflow/XAdyyoHUsbqPghUXvtvtocTXtH1q |
e38f618
to
93859a4
Compare
93859a4
to
cb3ad2e
Compare
cb3ad2e
to
87f5826
Compare
87f5826
to
1c39601
Compare
1c39601
to
cd475fa
Compare
cd475fa
to
93a23f7
Compare
typescript/web/src/components/labeling-tool/openlayers-map/openlayers-map.tsx
Outdated
Show resolved
Hide resolved
@nicOwlas Code is OK to be merged. Just let me know once you've finished our first announcement (of type |
The announcement is done. If you want to review it, it's here: https://announcekit.app/dashboard/post |
Since AnnounceKit filters doesn't seems to work yet: announcekitapp/announcekit-react#28
@nicOwlas I've reviewed the post: I'd add a warning-sign emoji ( @Jordanlelay I've just removed the announcement code from |
Good to go ! |
Work performed
I've made sure that AnnounceKit is used in both the website an in the app.
See #635 for details
Results
Users should be informed about our new features, discounts, downtimes and maintenance operations
Problems encountered
See my progress reports in #635
Caveats
Can't be used without an AnnounceKit account.
Resolved issues
Closes #635
Newly raised issues
No