Merge pull request #16 from which-ecosystem/feedback

eug-vs · web-flow · commit 88e97a42096c · 2020-06-28T19:01:52.000+03:00
Feedback endpoint &amp; schema updates
diff --git a/hooks/convertPoll.ts b/hooks/convertPoll.ts
diff --git a/hooks/signAuthority.ts b/hooks/signAuthority.ts
@@ -0,0 +1,8 @@
+import { HookContext } from '@feathersjs/feathers';
+
+export default async (context: HookContext): Promise<HookContext> => {
+  const { params: { user } } = context;
+  context.data.authorId = user._id;
+  return context;
+};
+
diff --git a/models/feedback/feedback.model.ts b/models/feedback/feedback.model.ts
@@ -0,0 +1,7 @@
+import { Model, model } from 'mongoose';
+import { FeedbackSchema, feedbackSchema } from './feedback.schema';
+
+feedbackSchema.index({ version: 1, authorId: 1 }, { unique: true }); // Unique together
+
+export default model<FeedbackSchema, Model<FeedbackSchema>>('Feedback', feedbackSchema);
+
diff --git a/models/feedback/feedback.schema.ts b/models/feedback/feedback.schema.ts
@@ -0,0 +1,28 @@
+import { Document, Schema, Types } from 'mongoose';
+
+export interface FeedbackSchema extends Document {
+  contents: string;
+  authorId: string;
+  score: number;
+  version: string;
+  createdAt: Date;
+}
+
+export const feedbackSchema = new Schema({
+  contents: String,
+  authorId: {
+    type: Types.ObjectId,
+    required: true,
+    ref: 'User'
+  },
+  score: {
+    type: Number,
+    required: true
+  },
+  version: {
+    type: String,
+    match: /^v\d+\.\d+\.\d+$/,
+    required: true
+  }
+}, { timestamps: true });
+
diff --git a/models/votes/vote.model.ts b/models/votes/vote.model.ts
@@ -1,7 +1,7 @@
 import { Model, model } from 'mongoose';
 import { VoteSchema, voteSchema } from './vote.schema';
 
-voteSchema.index({ pollId: 1, userId: 1 }, { unique: true }); // Unique together
+voteSchema.index({ pollId: 1, authorId: 1 }, { unique: true }); // Unique together
 
 export default model<VoteSchema, Model<VoteSchema>>('Vote', voteSchema);
 
diff --git a/models/votes/vote.schema.ts b/models/votes/vote.schema.ts
@@ -1,12 +1,10 @@
 import { Document, Schema, Types } from 'mongoose';
 import { Vote } from 'which-types';
 
-export interface VoteSchema extends Document, Omit<Vote, '_id'> {
-  password: string;
-}
+export interface VoteSchema extends Document, Omit<Vote, '_id'> {}
 
 export const voteSchema = new Schema({
-  userId: {
+  authorId: {
     type: Types.ObjectId,
     ref: 'user',
     required: true
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -33,7 +33,7 @@
     "mongoose": "^5.9.18",
     "ts-node": "^8.10.2",
     "typescript": "^3.9.5",
-    "which-types": "^1.4.2"
+    "which-types": "^1.6.1"
   },
   "repository": {
     "type": "git",
diff --git a/populateDb.ts b/populateDb.ts
@@ -1,7 +1,12 @@
 import mongoose from 'mongoose';
 import bluebird from 'bluebird';
 import _ from 'lodash';
-import { User, Poll, Vote } from 'which-types';
+import {
+  User,
+  Poll,
+  Vote,
+  Feedback
+} from 'which-types';
 
 import app from './app';
 
@@ -71,6 +76,13 @@ const createVote = (userId: string, pollId: string): Promise<Vote> => {
   }, { user: { _id: userId }, authenticated: true });
 };
 
+const createFeedback = (userId: string): Promise<Feedback> => {
+  return app.service('feedback').create({
+    version: 'v1.0.0',
+    score: _.sample([1, 2, 3, 4, 5]),
+    content: 'Absolutely amazing!'
+  }, { user: { _id: userId }, authenticated: true });
+};
 
 const populate = async () => {
   const users = await bluebird.map(names, name => createUser(name));
@@ -80,6 +92,10 @@ const populate = async () => {
     return createPoll(user?._id || '');
   });
 
+  await bluebird.map(users, user => {
+    return createFeedback(user?._id || '');
+  });
+
   await bluebird.map(users, user => {
     const pollsToVote = _.sampleSize(polls, _.random(0, POLLS_AMOUNT));
     return bluebird.map(pollsToVote, poll => createVote(user?._id || '', poll?._id || ''));
diff --git a/services/feed/feed.hooks.ts b/services/feed/feed.hooks.ts
@@ -7,7 +7,7 @@ const raiseNewVerifedPolls = async (context: HookContext): Promise<HookContext>
   // Raise unseen verified polls to the very top
   context.result = _.sortBy(
     context.result,
-    poll => !(poll.author.verified && !poll.userChoice)
+    poll => !(poll.author.verified && !poll.vote)
   );
   return context;
 };
@@ -16,7 +16,7 @@ const lowerOldPolls = async (context: HookContext): Promise<HookContext> => {
   // Move all seen polls down
   context.result = _.sortBy(
     context.result,
-    poll => !!poll.userChoice
+    poll => !!poll.vote
   );
   return context;
 };
diff --git a/services/feedback/feedback.hooks.ts b/services/feedback/feedback.hooks.ts
@@ -0,0 +1,9 @@
+import requireAuth from '../../hooks/requireAuth';
+import signAuthority from '../../hooks/signAuthority';
+
+export default {
+  before: {
+    create: [requireAuth, signAuthority]
+  }
+};
+
diff --git a/services/feedback/feedback.service.ts b/services/feedback/feedback.service.ts
@@ -0,0 +1,13 @@
+import { Application } from '@feathersjs/express';
+import service from 'feathers-mongoose';
+import Model from '../../models/feedback/feedback.model';
+
+import hooks from './feedback.hooks';
+
+const FeebackService = service({ Model });
+
+export default (app: Application): void => {
+  app.use('/feedback', FeebackService);
+  app.service('feedback').hooks(hooks);
+};
+
diff --git a/services/index.ts b/services/index.ts
@@ -5,6 +5,7 @@ import Profiles from './profiles/profiles.service';
 import Votes from './votes/votes.service';
 import Auth from './auth/auth.service';
 import Feed from './feed/feed.service';
+import Feedback from './feedback/feedback.service';
 
 import tryAuthenticate from '../hooks/tryAuthenticate';
 import logging from '../hooks/logging';
@@ -17,6 +18,7 @@ export default (app: Application): void => {
   app.configure(Profiles);
   app.configure(Votes);
   app.configure(Feed);
+  app.configure(Feedback);
 
   app.hooks({
     before: {
diff --git a/services/polls/polls.hooks.ts b/services/polls/polls.hooks.ts
@@ -1,6 +1,47 @@
-import convertPoll from '../../hooks/convertPoll';
+import { HookContext } from '@feathersjs/feathers';
+import { Types } from 'mongoose';
+import bluebird from 'bluebird'; import _ from 'lodash';
+import { Poll } from 'which-types';
+
+import { PollSchema } from '../../models/polls/poll.schema';
+import VoteModel from '../../models/votes/vote.model';
 import sortByDate from '../../hooks/sortByDate';
 
+
+const convertPoll = async (context: HookContext): Promise<HookContext> => {
+  const { app, result, params: { user } } = context;
+
+  const convert = async (poll: PollSchema): Promise<Poll | null> => {
+    const author = await app.service('users').get(poll.authorId);
+
+    const contents = await VoteModel.aggregate([
+      { $match: { pollId: Types.ObjectId(poll._id) } },
+      { $group: { _id: '$which', total: { $sum: 1 } } }
+    ]).then(groups => groups.reduce(
+      (acc, group) => _.set(acc, `${group._id}.votes`, group.total),
+      { left: { votes: 0 }, right: { votes: 0 } }
+    ));
+
+    const vote = await VoteModel.findOne(
+      { pollId: poll._id, authorId: user?._id }
+    );
+
+    return _.merge(
+      _.omit(poll, ['authorId']),
+      { author, contents, vote }
+    );
+  };
+
+  if (Array.isArray(result)) {
+    const polls = await bluebird.map(result, (poll: PollSchema) => convert(poll));
+    context.result = _.compact(polls);
+  } else {
+    context.result = await convert(result);
+  }
+  return context;
+};
+
+
 export default {
   before: {
     find: sortByDate
diff --git a/services/votes/votes.hooks.ts b/services/votes/votes.hooks.ts
@@ -1,15 +1,9 @@
-import { HookContext } from '@feathersjs/feathers';
 import requireAuth from '../../hooks/requireAuth';
-
-const addUserId = async (context: HookContext): Promise<HookContext> => {
-  const { params: { user } } = context;
-  context.data.userId = user._id;
-  return context;
-};
+import signAuthority from '../../hooks/signAuthority';
 
 export default {
   before: {
-    create: [requireAuth, addUserId]
+    create: [requireAuth, signAuthority]
   }
 };
 
