Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merge x/reports into x/posts #429

Closed
RiccardoM opened this issue May 18, 2021 · 0 comments · Fixed by #444
Closed

Merge x/reports into x/posts #429

RiccardoM opened this issue May 18, 2021 · 0 comments · Fixed by #444
Assignees
@leobragaz
Labels
kind/enhancement Enhance an already existing feature; no "New feature" to add quality/code-cleanliness Improves the overall quality of code x/posts Post module
Milestone
v0.16.3

Comments

@RiccardoM
Copy link
Contributor

Currently we have a module named x/reports to handle only posts reports. Since this is pretty easy to do, I think we should merge it directly inside the x/posts module directly
@RiccardoM RiccardoM added the kind/enhancement Enhance an already existing feature; no "New feature" to add label May 18, 2021
@RiccardoM RiccardoM added this to the v0.16.3 milestone May 18, 2021
@leobragaz leobragaz self-assigned this May 18, 2021
@RiccardoM RiccardoM added x/posts Post module quality/code-cleanliness Improves the overall quality of code labels May 25, 2021
@leobragaz leobragaz mentioned this issue May 27, 2021
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leobragaz leobragaz

Labels
kind/enhancement Enhance an already existing feature; no "New feature" to add quality/code-cleanliness Improves the overall quality of code x/posts Post module
Projects
None yet
Milestone
v0.16.3
Development

Successfully merging a pull request may close this issue.

Merge Reports inside Posts desmos-labs/desmos
2 participants
@RiccardoM @leobragaz