#436: sync notification with timeline
This commit is contained in:
parent
060d3b0713
commit
13e727b5ee
3 changed files with 19 additions and 12 deletions
|
@ -6,15 +6,15 @@
|
||||||
>
|
>
|
||||||
</status>
|
</status>
|
||||||
<div class="non-mention" :class="[userClass, { highlighted: userStyle }]" :style="[ userStyle ]"v-else>
|
<div class="non-mention" :class="[userClass, { highlighted: userStyle }]" :style="[ userStyle ]"v-else>
|
||||||
<a class='avatar-container' :href="notification.action.user.statusnet_profile_url" @click.stop.prevent.capture="toggleUserExpanded">
|
<a class='avatar-container' :href="notification.action.account.statusnet_profile_url" @click.stop.prevent.capture="toggleUserExpanded">
|
||||||
<UserAvatar :compact="true" :betterShadow="betterShadow" :src="notification.action.user.profile_image_url_original"/>
|
<UserAvatar :compact="true" :betterShadow="betterShadow" :src="notification.action.account.profile_image_url_original"/>
|
||||||
</a>
|
</a>
|
||||||
<div class='notification-right'>
|
<div class='notification-right'>
|
||||||
<UserCard :user="notification.action.user" :rounded="true" :bordered="true" v-if="userExpanded"/>
|
<UserCard :user="notification.action.account" :rounded="true" :bordered="true" v-if="userExpanded"/>
|
||||||
<span class="notification-details">
|
<span class="notification-details">
|
||||||
<div class="name-and-action">
|
<div class="name-and-action">
|
||||||
<span class="username" v-if="!!notification.action.user.name_html" :title="'@'+notification.action.user.screen_name" v-html="notification.action.user.name_html"></span>
|
<span class="username" v-if="!!notification.action.account.name_html" :title="'@'+notification.action.account.screen_name" v-html="notification.action.account.name_html"></span>
|
||||||
<span class="username" v-else :title="'@'+notification.action.user.screen_name">{{ notification.action.user.name }}</span>
|
<span class="username" v-else :title="'@'+notification.action.account.screen_name">{{ notification.action.account.name }}</span>
|
||||||
<span v-if="notification.type === 'like'">
|
<span v-if="notification.type === 'like'">
|
||||||
<i class="fa icon-star lit"></i>
|
<i class="fa icon-star lit"></i>
|
||||||
<small>{{$t('notifications.favorited_you')}}</small>
|
<small>{{$t('notifications.favorited_you')}}</small>
|
||||||
|
@ -30,18 +30,18 @@
|
||||||
</div>
|
</div>
|
||||||
<div class="timeago" v-if="notification.type === 'follow'">
|
<div class="timeago" v-if="notification.type === 'follow'">
|
||||||
<span class="faint">
|
<span class="faint">
|
||||||
<timeago :since="notification.action.created_at" :auto-update="240"></timeago>
|
<timeago :since="notification.action.notified_at" :auto-update="240"></timeago>
|
||||||
</span>
|
</span>
|
||||||
</div>
|
</div>
|
||||||
<div class="timeago" v-else>
|
<div class="timeago" v-else>
|
||||||
<router-link v-if="notification.status" :to="{ name: 'conversation', params: { id: notification.status.id } }" class="faint-link">
|
<router-link v-if="notification.status" :to="{ name: 'conversation', params: { id: notification.status.id } }" class="faint-link">
|
||||||
<timeago :since="notification.action.created_at" :auto-update="240"></timeago>
|
<timeago :since="notification.action.notified_at" :auto-update="240"></timeago>
|
||||||
</router-link>
|
</router-link>
|
||||||
</div>
|
</div>
|
||||||
</span>
|
</span>
|
||||||
<div class="follow-text" v-if="notification.type === 'follow'">
|
<div class="follow-text" v-if="notification.type === 'follow'">
|
||||||
<router-link :to="userProfileLink(notification.action.user)">
|
<router-link :to="userProfileLink(notification.action.user)">
|
||||||
@{{notification.action.user.screen_name}}
|
@{{notification.action.account.screen_name}}
|
||||||
</router-link>
|
</router-link>
|
||||||
</div>
|
</div>
|
||||||
<template v-else>
|
<template v-else>
|
||||||
|
|
|
@ -271,7 +271,12 @@ const addNewStatuses = (state, { statuses, showImmediately = false, timeline, us
|
||||||
}
|
}
|
||||||
|
|
||||||
const addNewNotifications = (state, { dispatch, notifications, older, visibleNotificationTypes }) => {
|
const addNewNotifications = (state, { dispatch, notifications, older, visibleNotificationTypes }) => {
|
||||||
|
const allStatuses = state.allStatuses
|
||||||
|
const allStatusesObject = state.allStatusesObject
|
||||||
each(notifications, (notification) => {
|
each(notifications, (notification) => {
|
||||||
|
notification.action = mergeOrAdd(allStatuses, allStatusesObject, notification.action).item
|
||||||
|
notification.status = notification.status && mergeOrAdd(allStatuses, allStatusesObject, notification.status).item
|
||||||
|
|
||||||
// Only add a new notification if we don't have one for the same action
|
// Only add a new notification if we don't have one for the same action
|
||||||
if (!state.notifications.idStore.hasOwnProperty(notification.id)) {
|
if (!state.notifications.idStore.hasOwnProperty(notification.id)) {
|
||||||
state.notifications.maxId = notification.id > state.notifications.maxId
|
state.notifications.maxId = notification.id > state.notifications.maxId
|
||||||
|
|
|
@ -180,8 +180,8 @@ export const parseStatus = (data) => {
|
||||||
// Missing!! fix in UI?
|
// Missing!! fix in UI?
|
||||||
// output.in_reply_to_screen_name = ???
|
// output.in_reply_to_screen_name = ???
|
||||||
|
|
||||||
// Not exactly the same but works
|
// It breaks the conversation when combined with notification
|
||||||
output.statusnet_conversation_id = data.id
|
// output.statusnet_conversation_id = data.id
|
||||||
|
|
||||||
if (output.type === 'retweet') {
|
if (output.type === 'retweet') {
|
||||||
output.retweeted_status = parseStatus(data.reblog)
|
output.retweeted_status = parseStatus(data.reblog)
|
||||||
|
@ -218,7 +218,6 @@ export const parseStatus = (data) => {
|
||||||
output.in_reply_to_status_id = data.in_reply_to_status_id
|
output.in_reply_to_status_id = data.in_reply_to_status_id
|
||||||
output.in_reply_to_user_id = data.in_reply_to_user_id
|
output.in_reply_to_user_id = data.in_reply_to_user_id
|
||||||
output.in_reply_to_screen_name = data.in_reply_to_screen_name
|
output.in_reply_to_screen_name = data.in_reply_to_screen_name
|
||||||
|
|
||||||
output.statusnet_conversation_id = data.statusnet_conversation_id
|
output.statusnet_conversation_id = data.statusnet_conversation_id
|
||||||
|
|
||||||
if (output.type === 'retweet') {
|
if (output.type === 'retweet') {
|
||||||
|
@ -266,6 +265,8 @@ export const parseFollow = (data) => {
|
||||||
output.visibility = true
|
output.visibility = true
|
||||||
output.created_at = new Date(data.created_at)
|
output.created_at = new Date(data.created_at)
|
||||||
output.user = parseUser(data.account)
|
output.user = parseUser(data.account)
|
||||||
|
output.notified_at = output.created_at
|
||||||
|
output.account = output.user
|
||||||
|
|
||||||
return output
|
return output
|
||||||
}
|
}
|
||||||
|
@ -286,7 +287,8 @@ export const parseNotification = (data) => {
|
||||||
: parseStatus(data.status)
|
: parseStatus(data.status)
|
||||||
if (data.type === 'reblog' || data.type === 'favourite') {
|
if (data.type === 'reblog' || data.type === 'favourite') {
|
||||||
output.status.user = parseUser(data.account)
|
output.status.user = parseUser(data.account)
|
||||||
output.status.created_at = new Date(data.created_at)
|
output.status.account = parseUser(data.account)
|
||||||
|
output.status.notified_at = new Date(data.created_at)
|
||||||
}
|
}
|
||||||
output.action = output.status // not sure
|
output.action = output.status // not sure
|
||||||
output.from_profile = parseUser(data.account)
|
output.from_profile = parseUser(data.account)
|
||||||
|
|
Loading…
Reference in a new issue