Wolfgang Huß
68c25b8d54
Merge branch 'master' of https://github.com/Human-Connection/Human-Connection into 1062-notification-about-comment-on-post
...
# Conflicts:
# backend/src/middleware/handleNotifications/handleNotifications.spec.js
2019-08-19 08:42:00 +02:00
Alina Beck
0aa1b8046b
remove grid logic from post component
2019-08-18 18:28:48 +01:00
Alina Beck
76be960c03
move grid calculations to masonry grid components
2019-08-18 18:25:14 +01:00
Wolfgang Huß
f653bac9e1
Merge branch 'master' of https://github.com/Human-Connection/Human-Connection into 1017-send-out-notifications-on-create-omment
...
# Conflicts:
# backend/src/middleware/handleHtmlContent/handleContentData.spec.js
2019-08-16 18:22:36 +02:00
roschaefer
aa52587f83
Get rid of property warnings
...
If we always choose `0` as the default value for `endVal` in case it is
not given (maybe apollo assigns null when the request is in flight) then
just make `0` the default.
2019-08-16 17:08:00 +02:00
Wolfgang Huß
efb5c75c24
Refactor after Roberts suggestions, number I
...
Start writing tests
2019-08-16 16:56:14 +02:00
mattwr18
90e5579cab
Merge pull request #1278 from Human-Connection/fix_hashtag_db_import
...
Fix hashtag db import
2019-08-16 14:37:09 +02:00
Matt Rider
dbaa8e687f
Fix vue warnings, fix tests, refactor
...
- was throwing an error when trying to update commentsCount because of new implementation by @roschaefer which uses countResolver, but there was no related for commentsCount, it was r... also commentsCount is no longer needed anywhere in the code base, it is commentedCount now
2019-08-16 13:28:55 +02:00
Matt Rider
d511d6aa78
Refactor graphql queries
...
- Remove duplicate queries
- Use smart query in pages/post/_id/_slug/index.vue to avoid multiple db requests for a post and its comments.
We cannot update the apollo cache with asyncData and smart queries have a prefetch policy set to true by default, which
means that they will resolve in a similar timeframe. https://stackoverflow.com/questions/55885337/in-nuxt-should-i-use-asyncdata-or-default-apollo-queries
https://vue-apollo.netlify.com/api/smart-query.html#options
https://vue-apollo.netlify.com/guide/ssr.html#vue-cli-plugin
2019-08-16 11:25:53 +02:00
roschaefer
51b734275a
The name of the hashtag is now it's ID
...
`name` was just a redundant attribute
2019-08-16 01:49:29 +02:00
roschaefer
3d4724e314
Fix Cannot read property countUser of null
...
This commit message is a great example of why you should explain
**the reason** of your commit.
When I came across this bug I had a quick look into the code where it
came from. I could see that the bug only happened after a timer and
apparently reverting the biggest part of d84892930295dcfd3f6687fc33c7234446127099
would fix the bug. However I have no idea what the following commit
message means:
```
commit d84892930295dcfd3f6687fc33c7234446127099
Author: Grzegorz Leoniec <greg@app-interactive.de>
Date: Wed Mar 6 18:45:57 2019 +0100
Improved countTo component
```
I just don't know why the code is there - like what is it's purpose
@appinteractive? I can only guess: I believe that it's supposed to
update the counters in-place (without starting from 0 everytime).
Because apollo was set to poll the data every second.
Taking that into account I would rather remove this polling feature
completely and have less code and less complexity. Admins can still
refresh the page.
2019-08-16 00:28:13 +02:00
Alina Beck
b05599de83
fix item height calculation
2019-08-15 21:44:22 +01:00
Alina Beck
da8491a559
style filter menu grid item
2019-08-15 21:29:37 +01:00
Alina Beck
8ad1ad3769
look at this new and shiny grid component
2019-08-15 21:06:26 +01:00
Wolfgang Huß
8acccc99d0
Implemented a reason in the Notification
...
Used this for displaying in the mentions menu in frontend.
Rewrite backend test.
2019-08-15 19:29:06 +02:00
Wolfgang Huß
6ab9a3204b
Fix the backend test
2019-08-15 11:07:57 +02:00
Alina Beck
38100b2df7
add masonry grid
2019-08-15 11:04:30 +02:00
Wolfgang Huß
0d66cc4dcf
Merge branch 'master' of https://github.com/Human-Connection/Human-Connection into 1017-send-out-notifications-on-create-omment
...
# Conflicts:
# backend/src/middleware/handleHtmlContent/handleContentData.js
# backend/src/schema/types/schema.gql
2019-08-15 10:16:33 +02:00
Wolfgang Huß
c2357d32b3
Fixed broken Editor tests without breaking the Mentions and the Hashtags in the Editor
2019-08-15 07:52:49 +02:00
Alexander Friedland
048c4a2502
Merge pull request #1224 from Human-Connection/A-1208-Registration-without-user-condition-confirmation
...
🍰 Registration confirmation of terms of use added II
2019-08-14 18:09:07 +02:00
Wolfgang Huß
43c2764fcf
Fix mentions and hashtags in editor
2019-08-14 18:05:03 +02:00
Robert Schäfer
b6c96b7a57
Merge pull request #1261 from Human-Connection/1201-localise-categories
...
Localise categories
2019-08-14 17:44:21 +02:00
roschaefer
58a7e1ce2d
Simplifycomponent, improve texts
...
@ogerly there is a difference between read and confirm: You can read but
don't have to confirm the terms and conditions. Therefore it's quite
important that the user also confirms the terms and conditions.
2019-08-14 16:38:35 +02:00
Matt Rider
f2dcd99ac5
Fix lint, remove console.log
2019-08-14 14:33:55 +02:00
ogerly
397b41377a
delete console.log
2019-08-14 14:31:47 +02:00
ogerly
a5f448036c
Suggestions implemented II
2019-08-14 14:27:44 +02:00
Matt Rider
ee93efd6d9
Localise categories
2019-08-14 13:57:39 +02:00
Wolfgang Huß
825a5f235a
Merge branch 'master' of https://github.com/Human-Connection/Human-Connection into 1017-send-out-notifications-on-create-omment
2019-08-14 13:10:19 +02:00
Robert Schäfer
7f509b3201
Implement block/unbock UI
2019-08-13 11:05:29 +02:00
Robert Schäfer
0e707cdd4c
Massive refactoring
...
Fix #1231 the filter menu once and for all...
2019-08-10 03:41:38 +02:00
Robert Schäfer
cea206d118
Merge pull request #1225 from Human-Connection/dependabot/npm_and_yarn/webapp/vue-sweetalert-icons-4.2.0
...
Bump vue-sweetalert-icons from 4.1.0 to 4.2.0 in /webapp
2019-08-09 14:13:55 +02:00
Robert Schäfer
5d57f7b376
No global import - Thank you @JorgenVatle! ❤️
...
In version `v4.2.0` of `vue-sweetalert-icons` the local import is back, see:
https://github.com/JorgenVatle/vue-sweetalert-icons/issues/3#issuecomment-519497796
2019-08-09 13:21:54 +02:00
Robert Schäfer
3261237b48
Merge pull request #1169 from Human-Connection/964-filter-posts-by-followers
...
Move filter by followers functionality to filter dropdown
2019-08-09 11:57:17 +02:00
ogerly
d545e84ce9
edit spec green
2019-08-09 07:27:38 +02:00
ogerly
f2dcb54d05
edit spec
2019-08-09 07:23:47 +02:00
Robert Schäfer
2eaf66378a
Merge pull request #1133 from Human-Connection/384-emotions-on-posts
...
Emotions on posts
2019-08-09 00:37:38 +02:00
Robert Schäfer
963cbbef32
RemovePostEmotions should return deleted object
...
@mattwr18 I prefer (I believe it's even best practice) that a delete
mutation should return the deleted object. If you run the delete
mutation again, it should return `null` because there is no object like
that anymore. That way the client knows if a delete mutation has changed
any state in the database.
Also I fixed another bug in the resolver. If your graphql mutation looks
like this:
```gql
mutation {
RemovePostEmotions(to:{ id:"p15"}, data:{emotion: angry}) {
from {
id
name
}
to {
id
title
}
emotion
}
}
```
Then you get errors because your resolver does not return the name for
the user or the title for the post anymore. Just use spread operator...
and it's fixed.
2019-08-08 23:51:26 +02:00
ogerly
d1830f5bed
add ceckbox, confirmed terms of use
2019-08-08 17:46:04 +02:00
Matt Rider
1930e5bafe
Add some component tests, rename files
...
- with new naming schema
2019-08-08 17:04:23 +02:00
Matt Rider
ced676d315
Merge branch 'master' of github.com:Human-Connection/Human-Connection into 1017-send-out-notifications-on-create-omment
2019-08-08 14:06:45 +02:00
Matt Rider
7c6dbd6351
Fix failing tests, remove comments
...
- Co-authored-by: Wolfgang Huß <wolle.huss@pjannto.com>
2019-08-08 14:03:59 +02:00
Matt Rider
21aa729edb
Merge branch 'master' of github.com:Human-Connection/Human-Connection into 1017-send-out-notifications-on-create-omment
2019-08-08 09:10:18 +02:00
Matt Rider
0cee0ecccb
Fix bug, maintain filters across page changes
...
- filter was a bit off for categories, clear the filter if there are no categories
- a user should maintain their filter until they clear it
2019-08-08 08:32:43 +02:00
Matt Rider
c00ab4450b
Merge branch 'master' of github.com:Human-Connection/Human-Connection into 964-filter-posts-by-followers
2019-08-08 07:24:13 +02:00
Matt Rider
faa4148858
Stub sweetalert-icon in tests which use
2019-08-07 15:20:36 +02:00
Matt Rider
39a94a8aac
Refactor
...
- follow PR review by @roschaefer
2019-08-07 14:55:32 +02:00
Robert Schäfer
3eab519760
Reduce the size of the emotion buttons
...
In my browser, the icons are so large that they overlap with the texts
below. I think a less intrusive size does the job equally well.
2019-08-06 22:37:36 +02:00
Matt Rider
1c0112e9d8
Merge branch 'master' of github.com:Human-Connection/Human-Connection into 384-emotions-on-posts
2019-08-06 19:00:14 +02:00
Robert Schäfer
27163b8d79
Install vue-sweetalert-icon globally
...
Updating the package to `v4.0.0` still gives me this warning:
```
WARN Compiled with 1 warnings
WARN in ./components/PasswordReset/ChangePassword.vue?vue&type=script&lang=js&
"export 'SweetalertIcon' was not found in 'vue-sweetalert-icons'
```
According to this thread https://github.com/JorgenVatle/vue-sweetalert-icons/issues/3
I copy+pasted the example configuration:
https://github.com/JorgenVatle/vue-sweetalert-icons-preview/blob/master/nuxt.config.js
https://github.com/JorgenVatle/vue-sweetalert-icons-preview/blob/master/plugins/vue-sweetalert-icons.js
And now the compile warnings are gone. I'm going to ask the maintainer if the
local import is not possible with `v4.0.0` anymore.
2019-08-06 11:03:12 +02:00
Matt Rider
f3dc5c6a68
Merge branch 'master' of github.com:Human-Connection/Human-Connection into 964-filter-posts-by-followers
2019-08-06 10:31:29 +02:00