Skip to content
New issue

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

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

Already on GitHub? # to your account

Remove posts helper #412

Merged
merged 1 commit into from
Oct 18, 2018
Merged

Remove posts helper #412

merged 1 commit into from
Oct 18, 2018

Conversation

enricostano
Copy link
Contributor

@enricostano enricostano commented Aug 22, 2018

WAT

"I love the smell of deleted code in the morning".

Test

Test everything related to (especially links and buttons):

  • Post
  • Tag

GIF

Copy link
Collaborator

@sauloperez sauloperez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤘

@enricostano
Copy link
Contributor Author

Deployed to staging

@sseerrggii
Copy link
Contributor

I can't edit Post, but I'm not sure if is it something fixed in other PR, I remember we fixed same problem some weeks ago...

peek 17-10-2018 16-48

@enricostano
Copy link
Contributor Author

I'll rebase it.

@enricostano enricostano force-pushed the feature/remove-posts-helper branch from 7de19ac to 4be234a Compare October 17, 2018 15:09
@enricostano
Copy link
Contributor Author

Rebased and deployed to staging @sseerrggii

@sseerrggii
Copy link
Contributor

Works fine ❤️!! Thanks

@enricostano enricostano merged commit 8bb7d99 into develop Oct 18, 2018
@enricostano enricostano deleted the feature/remove-posts-helper branch October 18, 2018 11:52
@enricostano enricostano mentioned this pull request Oct 19, 2018
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants