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.

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

feat(webapp): alternative solution for filter and order posts #6367

Merged
merged 51 commits into from
Jun 5, 2023

Conversation

Mogge
Copy link
Contributor

@Mogge Mogge commented Jun 2, 2023

🍰 Pullrequest

Issues

  • None

Todo

  • None

Tirokk and others added 30 commits May 26, 2023 19:21
- Unset post type filter menu button by clicking a set button.
…lot-Social into 6190-extend-newsfeed-filter-menu-for-events
…lot-Social into 6190-extend-newsfeed-filter-menu-for-events
…cial into 6190-extend-newsfeed-filter-menu-for-events
Co-authored-by: Hannes Heine <heine.hannes@gmail.com>
Co-authored-by: Hannes Heine <heine.hannes@gmail.com>
Co-authored-by: Hannes Heine <heine.hannes@gmail.com>
Co-authored-by: Hannes Heine <heine.hannes@gmail.com>
webapp/store/posts.spec.js Outdated Show resolved Hide resolved
Copy link
Member

@Tirokk Tirokk left a comment

Choose a reason for hiding this comment

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

Even if I'm not allowed to do so by our rules …

I give you my approve and thank you that you made the store lighter and much more clearly as well as the tests slim. 🙏🏼 🚀🚀💫💫
I love it! 😍

Would be nice if you give a good hint here in the description to my PR #6341 and what it has todo with mine.
And I will do as well on mine if I close it.

PS: For transparence of our talk and because I feel I need to express my feelings to the team.

My PR came late. Sorry for that, but I was prepared to work on the weekend.

Like I said:
My PR was in a prereview state, because I was not satisfied with it and liked to go ahaead working on it. I got not the chance to better and improve my code myself and have a learning by doing or by having a pairing with you.
Because you throw my tests completetly I lost a half day of my work which makes me bad feelings as well.
I would've liked better if you have talked to me before you worked on my issue and code silently.
I'm sure we would have found a solution and you could have worked on my PR, instead of open a new one. That gave me the impression that I just do bullshit and that we are more in a kind of competition, even if you wrote on Discord that we are not.
Thanks for your explanation, your excuse, and for expressing your worries todo so in our talk.
And that we all will try todo better in the future.
I like always to learn and be very interested to have a good code base, that both is my main interest.

@Mogge Mogge merged commit befc37f into master Jun 5, 2023
20 checks passed
@Mogge Mogge deleted the alternate-solution-filter-order-post-types branch June 5, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants