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 recipes for freelancers #534

Merged
merged 8 commits into from
May 9, 2024
Merged

add recipes for freelancers #534

merged 8 commits into from
May 9, 2024

Conversation

sakarimov
Copy link
Contributor

@sakarimov sakarimov commented Apr 26, 2024

  • add recipes for freelancers
  • edit recipes for freelancers after test

Pre-flight Checklist

Please ensure you've completed all of the following.

Adding new recipe

Service: services for freelancers, it contains various freelancers platform, i intended to add more of if, but for now i already have 3 of those

URL: guru.com, truelancer.com, peopleperhour.com

Service ID: guru.com, truelancer, peopleperhour

Terminal output

> ferdium-recipes@1.0.1 package /run/media/officestorage/karya/contrib/github/ferdium-recipes
> node scripts/package.js

✅ Successfully packaged and added 348 recipes (0 unsuccessful recipes)

Additional information

  • I am the original creator of this package
  • I have run the pnpm lint:fix && pnpm reformat-files && pnpm package and verified that there are no validation errors reported for this package
  • My recipe has been tested to work inside Ferdium

Copy link
Member

@SpecialAro SpecialAro left a comment

Choose a reason for hiding this comment

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

Hi @sakarimov, thank you for your contribution.

Please fix my review comments whenever you can 🙏

Thank you!

.npmrc Outdated Show resolved Hide resolved
recipes/peopleperhour/package.json Outdated Show resolved Hide resolved
recipes/guru/package.json Outdated Show resolved Hide resolved
recipes/truelancer/package.json Outdated Show resolved Hide resolved
recipes/truelancer/icon.svg Outdated Show resolved Hide resolved
recipes/guru/icon.svg Outdated Show resolved Hide resolved
recipes/peopleperhour/icon.svg Outdated Show resolved Hide resolved
@sakarimov
Copy link
Contributor Author

everything resolved, thank u for the guidance @SpecialAro

recipes/peopleperhour/webview.js Outdated Show resolved Hide resolved
recipes/guru/webview.js Show resolved Hide resolved
recipes/truelancer/webview.js Show resolved Hide resolved
@sakarimov
Copy link
Contributor Author

i want to resolve issue on ferdium/ferdium-app#1742 to add recipe of upwork should i create new pr or can i just add the recipe here? @vraravam @SpecialAro

@vraravam
Copy link
Contributor

vraravam commented May 9, 2024

i want to resolve issue on ferdium/ferdium-app#1742 to add recipe of upwork should i create new pr or can i just add the recipe here? @vraravam @SpecialAro

pls create a new PR

@vraravam vraravam merged commit f90e958 into ferdium:main May 9, 2024
2 checks passed
@vraravam
Copy link
Contributor

vraravam commented May 9, 2024

@all-contributors please add @sakarimov for code

Copy link
Contributor

@vraravam

I've put up a pull request to add @sakarimov! 🎉

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.

3 participants