Skip to content

Proposals v2. Discussion system. Iteration5#176

Closed
shamil-gadelshin wants to merge 6 commits intoJoystream:proposals_v2from
shamil-gadelshin:proposals_v2_iteration5
Closed

Proposals v2. Discussion system. Iteration5#176
shamil-gadelshin wants to merge 6 commits intoJoystream:proposals_v2from
shamil-gadelshin:proposals_v2_iteration5

Conversation

@shamil-gadelshin
Copy link
Contributor

@shamil-gadelshin shamil-gadelshin commented Mar 4, 2020

Features:

  • spam-gate: max allowed post count in a row
  • modification of the post and thread author authentication
  • discussion events

Work to do after the Rome-release:

Add ‘max number of threads by same author in a row’ limit
- add ActorOriginValidator trait
- update create_thread() with ThreadAuthorOriginValidator
- upgrade add_post() and update_post() with PostAuthorOriginValidator
- add ThreadCreated, PostCreated, PostUpdated events
- add tests
- move error messages to the separate module
- add ThreadAuthorId param stub
- update mock and tests
@shamil-gadelshin shamil-gadelshin requested a review from bedeho March 4, 2020 07:28
@shamil-gadelshin
Copy link
Contributor Author

Will be reworked during iteration 6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant