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(Teckie): add Teckie support #9280

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Conversation

bakzkndd
Copy link
Contributor

@bakzkndd bakzkndd commented Feb 13, 2025

Description

Add support for Teckie. A dutch computer sales, repair and support company. Has support for all pages. Checkout page just shows the status for the base url to protect privacy.
Sidenote: didn't run yarn format but instead npm run lint:fix. yarn format didn't work for me and was told in the Discord server to use npm run lint:fix instead. Not sure if the PR template needs updating.

Acknowledgements

Screenshots

Proof showing the creation/modification is working as expected Example 1: ![image](https://github.com/user-attachments/assets/82b7052e-db4d-43ec-8763-6570b1568120) ![image](https://github.com/user-attachments/assets/4d944f8c-f558-43cf-a2e1-2561a6de8157)

Example 2:
image
{65482BDE-C5B9-49D8-AAB4-572921A8C4AB}
{EDBFE21A-70CD-4A95-8F1E-BCD94CE958EF}

@bakzkndd bakzkndd requested a review from Kuriel23 February 13, 2025 18:30
Copy link
Contributor

@darkvillager2 darkvillager2 left a comment

Choose a reason for hiding this comment

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

For English see below.
---------
Hallo Maatje,

Bedankt voor je PR. Ik heb wat opmerkingen/vragen hierover. Graag de verzochte wijzigingen maken.

Mocht je vragen hebben, stel ze in dit PR of op discord (@dark_ville)

Groet,
Mede Kaaskop
Dark
-----------
Hello Mate,

Thanks for your PR, i have some questions/notes about it. Please make the changes as requested.

If you have any questions feel free to ask in this pr or on discord (@dark_ville)

~Dark

@bakzkndd
Copy link
Contributor Author

Hello from mobile, I've been and still am sick with the flu. Will make changes once I'm better.

Bas950 and others added 5 commits February 20, 2025 14:44
Co-authored-by: github plz bring back DarkVIllager <[email protected]>
Signed-off-by: bakzkndd <[email protected]>
Co-authored-by: github plz bring back DarkVIllager <[email protected]>
Signed-off-by: bakzkndd <[email protected]>
Co-authored-by: github plz bring back DarkVIllager <[email protected]>
Signed-off-by: bakzkndd <[email protected]>
Co-authored-by: github plz bring back DarkVIllager <[email protected]>
Signed-off-by: bakzkndd <[email protected]>
realized that all the cases could get their info from the same place, so to reduce double code just moved it to the bottom
@bakzkndd
Copy link
Contributor Author

oh no I forgot the good commit names

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

Successfully merging this pull request may close these issues.

6 participants