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

Add Skroutz engineering to the list of users #1798

Merged
merged 2 commits into from
Jul 12, 2023
Merged

Conversation

nitsas
Copy link
Contributor

@nitsas nitsas commented Jul 12, 2023

We've started using ViewComponent since March 2023 and we are extremely satisfied with the experience!

What are you trying to accomplish?

Added Skroutz to the list of ViewComponent users, with a link to our engineering blog. 🙂

What approach did you choose and why?

Hi, I read CONTRIBUTING.md but didn't add myself in the list of contributors because I only added a new entry in the list of users of the library. I didn't actually contribute anything else. I'll check the issues and try to help in the future. 🙂

Anything you want to highlight for special attention from reviewers?

We've started using ViewComponent since March 2023 and we are extremely satisfied with the experience!
@nitsas
Copy link
Contributor Author

nitsas commented Jul 12, 2023

Test capture_patch_enabled fails, but I'm not sure how it's related to my changes. 😕

@Spone Spone merged commit 095f76f into ViewComponent:main Jul 12, 2023
18 of 19 checks passed
claudiob pushed a commit to claudiob/view_component that referenced this pull request Dec 22, 2023
* Add Skroutz engineering to the list of users

We've started using ViewComponent since March 2023 and we are extremely satisfied with the experience!

* fixup! Add Skroutz engineering to the list of users
claudiob pushed a commit to claudiob/view_component that referenced this pull request Jan 3, 2024
* Add Skroutz engineering to the list of users

We've started using ViewComponent since March 2023 and we are extremely satisfied with the experience!

* fixup! Add Skroutz engineering to the list of users
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.

2 participants