7083 Commits

Author SHA1 Message Date
mattwr18
f2e2c78582 Add tests for notifications index page 2019-11-11 08:52:51 +01:00
mattwr18
64e2a2a7c2 Add story for Paginate component 2019-11-11 08:52:51 +01:00
mattwr18
7d0b9861b8 Add story for AvatarMenu 2019-11-11 08:52:51 +01:00
mattwr18
49f73976b7 Fix DropdownFilter story after refactor
- it was refactored to make reuse easier
2019-11-11 08:52:04 +01:00
mattwr18
d56c81dc3d Add test for NotificationsTable and refactor 2019-11-11 08:52:04 +01:00
mattwr18
09e4c20bb8 Update test to findIndex instead of hard coding
Co-authored-by: Mike Aono <aonomike@gmail.com>
2019-11-11 08:52:04 +01:00
mattwr18
b7e39ca3db Add test for DropdownComponent 2019-11-11 08:52:04 +01:00
mattwr18
088d4a8ae5 Use verbose variables 2019-11-11 08:52:04 +01:00
mattwr18
ddb9fcdcb7 Make dropdown filter reusable 2019-11-11 08:52:04 +01:00
mattwr18
780a418254 Fix lint 2019-11-11 08:52:04 +01:00
roschaefer
25c9b9585b Translate dropdown values on notification page
@mattwr18 this solves the problem by 50%. If you switch the language and
click on the notifications dropdown filter, you will see the translated
labels. However, the label showing the selected value will not change,
probably because `<ds-menu>` caches the selected option 😞.
2019-11-11 08:52:04 +01:00
roschaefer
8e129c6001 fix: update counter when notification is read
@mattwr18 vue-apollo rocks! Taking the time to study the docs is a
rewarding investment.

My first idea was to cache the `unreadNotificationsCount` with Vuex.
But the docs of apollo even suggest to use apollo's local state as a
complete replacement of Vuex:
https://vue-apollo.netlify.com/guide/local-state.html

Then I further investigated why the updated `NOTIFIED` objects won't
update the notification counter. Turns out: They don't have an ID and
the computed property didn't fire when the notifications array would
change. I fixed both in this commit and yes, it works as expected.

No additional code required 💪
2019-11-11 08:52:04 +01:00
roschaefer
8ca2445a25 Fix data of NotificationsTable.story.js
@mattwr18 probably you wanted to override the `author` of the
comment/post, right? If you use the { ... desctructuring } operator,
whatever comes last takes precedence. So you probaby want to put your
override last?
2019-11-11 08:52:04 +01:00
mattwr18
6fca753c1f Add component test for Empty.vue 2019-11-11 08:52:04 +01:00
mattwr18
1d8e328a47 Clean up
- remove unused data variable
- avoid scoped scss
2019-11-11 08:52:04 +01:00
mattwr18
715261238a Protect against cypher injection vulnerability
- following @roschaefer PR review suggestion
2019-11-11 08:52:04 +01:00
mattwr18
7564908456 Refactor to follow @alina-beck suggestions
- Update CSS
- Use design tokens where possible
- Update spec description
- Avoid scoped scss as it's more expensive
- did not change the next and back button as they weren't really visible
without the primary class
- Fix German translation
2019-11-11 08:52:04 +01:00
mattwr18
44ed462db2 Add NotificationsTable story, refactor code
- Use percentages for width on all columns to help with resonsiveness
- export constants from other stories to dry code out
2019-11-11 08:52:04 +01:00
mattwr18
542647f418 Add NotificationsDropdownFilter story, refactor
- rename incorrectly named varaibles
- use filterOptions to remove unneccessary computed routes

Co-authored-by: @alina-beck
2019-11-11 08:52:04 +01:00
mattwr18
3ae71544d6 Add story for Empty component
Co-authored-by: @alina-beck
2019-11-11 08:52:04 +01:00
mattwr18
a1af4f5037 Add storybook notes
Co-authored-by: @alina-beck
2019-11-11 08:52:04 +01:00
mattwr18
aabaa2e637 Fix typo
Co-authored-by: Alina Beck <alina.beck@mail.com>
2019-11-11 08:50:35 +01:00
mattwr18
b916c9fe83 Add portuguese translations
Co-authored-by: ppelegrin <56610168+ppelegrin@users.noreply.github.com>
2019-11-11 08:50:35 +01:00
mattwr18
cb2cc37556 Extract components following @alina-beck suggestion
- see https://github.com/Human-Connection/Human-Connection/pull/2075/files
2019-11-11 08:50:35 +01:00
mattwr18
7007b1c444 Paginate notifications, reusable component w/ test 2019-11-11 08:50:35 +01:00
mattwr18
084388a21e Follow new file structure
- avoid nested directories
2019-11-11 08:50:35 +01:00
mattwr18
f34c8f44e0 Update resolver for paginated notifications 2019-11-11 08:50:35 +01:00
mattwr18
4448c12f2e Add componenet test for AvatarMenu
Co-authored-by: mindcodemediator
<mindcodemediator@users.noreply.github.com>
Co-authored-by: Mike Aono <aonomike@gmail.com>
2019-11-11 08:50:35 +01:00
mattwr18
d83a424513 Add styling
- to link in NotificationMenu
- make table more mobile responsive
- remove unneccessary props

Co-authored-by: Alina Beck <alina.beck@mail.com>
2019-11-11 08:50:35 +01:00
mattwr18
994a0b049d Extract AvatarMenu into its own component 2019-11-11 08:50:35 +01:00
mattwr18
58b1f7948c Follow @alina-beck and @Tirokk suggestions
- following PR reviews
2019-11-11 08:50:35 +01:00
mattwr18
e8cf51095a Add back missing translation 2019-11-11 08:50:35 +01:00
mattwr18
cf8124fb2d Clean up/add german translations 2019-11-11 08:50:35 +01:00
mattwr18
b79770469f Update notifications page
- display more notifications by simplifying the table
- get sorting working
2019-11-11 08:50:35 +01:00
mattwr18
7cdc12f4b9 Add notifications page with Notifications in table
- At the moment, the notifications are displayed in a table, and have
the same functionality as the NotificationMenu where you can click on
the title of the Post and visit the Post

- Unsure about the styling, it would kind of be nice to have a more
condensed list to see more notifications per page, but still be able to
click on a row and visit the Post
2019-11-11 08:50:35 +01:00
mattwr18
ceb77d2bb6
Merge pull request #1909 from Human-Connection/add-missing-portuguese-translations
Add missing portuguese translation
2019-11-11 08:49:19 +01:00
Alina Beck
719c978603
Apply suggestions from code review
Co-Authored-By: Robert Schäfer <git@roschaefer.de>
2019-11-11 10:13:10 +03:00
Alina Beck
263ee16e3d
Merge pull request #2159 from Human-Connection/migrate-design-tokens
Migrate design tokens
2019-11-11 10:06:10 +03:00
Vasily Belolapotkov
dc09240c20 sync with 'master' to cleanup PR 2019-11-10 11:48:39 +03:00
roschaefer
7c278d7c7f fix: send 403 if you try to edit another's post
This also fixes the glitch that you click on "Edit post" and the
contribution form is empty.
2019-11-08 17:30:29 +01:00
roschaefer
6c947f782d Fix CSS classes 2019-11-08 17:30:05 +01:00
roschaefer
4108bb7d71 Merge remote-tracking branch 'origin/2119_Create_Post_consistent_form_input_validation' into 2119_Create_Post_consistent_form_input_validation-improvements 2019-11-08 16:32:16 +01:00
dependabot-preview[bot]
4834556623
build(deps-dev): bump cypress-file-upload from 3.4.0 to 3.5.0
Bumps [cypress-file-upload](https://github.com/abramenal/cypress-file-upload) from 3.4.0 to 3.5.0.
- [Release notes](https://github.com/abramenal/cypress-file-upload/releases)
- [Commits](https://github.com/abramenal/cypress-file-upload/compare/v3.4.0...v3.5.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
2019-11-08 04:21:08 +00:00
dependabot-preview[bot]
fe2beaeedc
build(deps): bump date-fns from 2.6.0 to 2.7.0 in /backend
Bumps [date-fns](https://github.com/date-fns/date-fns) from 2.6.0 to 2.7.0.
- [Release notes](https://github.com/date-fns/date-fns/releases)
- [Changelog](https://github.com/date-fns/date-fns/blob/master/CHANGELOG.md)
- [Commits](https://github.com/date-fns/date-fns/compare/v2.6.0...v2.7.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
2019-11-08 04:21:05 +00:00
dependabot-preview[bot]
a47e495537
build(deps-dev): bump date-fns from 2.6.0 to 2.7.0
Bumps [date-fns](https://github.com/date-fns/date-fns) from 2.6.0 to 2.7.0.
- [Release notes](https://github.com/date-fns/date-fns/releases)
- [Changelog](https://github.com/date-fns/date-fns/blob/master/CHANGELOG.md)
- [Commits](https://github.com/date-fns/date-fns/compare/v2.6.0...v2.7.0)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
2019-11-08 04:20:40 +00:00
roschaefer
a2dbe47f1f Fix tests 2019-11-08 00:50:20 +01:00
roschaefer
62b393b0f0 Refactoring of Categories Select component 2019-11-07 23:23:47 +01:00
ogerly
dd03f4b7f0 Fix Add Language check for form submit 2019-11-07 15:40:58 +01:00
roschaefer
c83a808825 Refactor content length validation with transform 2019-11-07 14:22:05 +01:00
roschaefer
253032468f Refactor: Re-use validation via error slot 2019-11-07 14:17:22 +01:00