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

Documentation #146

Merged
merged 19 commits into from
May 16, 2024
Merged

Documentation #146

merged 19 commits into from
May 16, 2024

Conversation

chemik-bit
Copy link
Collaborator

Readme update, gmail tool documentation (calendly documentation will be updated in next commit), frontend fix @iljamak have a look at the updates :)

Copy link

vercel bot commented May 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sales-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2024 9:18pm
sales-gpt-api ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2024 9:18pm
sales-gpt-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 15, 2024 9:18pm

@filip-michalsky
Copy link
Owner

nice!

@filip-michalsky filip-michalsky merged commit a715ead into main May 16, 2024
9 checks passed
@iljamak
Copy link
Collaborator

iljamak commented May 17, 2024

Everything seems to be good

@filip-michalsky
Copy link
Owner

@iljamak did you:
A) run the unit tests
B) spin up the frontend locally without container
C) spin up front end in docker locally
D) tested all the tools in th PR
E) reviewed the documentation provided and followed it and were able to reproduce results with all of the tools?

Let me know if you did all A thru E?

@chemik-bit
Copy link
Collaborator Author

@iljamak did you: A) run the unit tests B) spin up the frontend locally without container C) spin up front end in docker locally D) tested all the tools in th PR E) reviewed the documentation provided and followed it and were able to reproduce results with all of the tools?

Let me know if you did all A thru E?

@iljamak I still have a few touchups to finish for perfect implementation. I will let you know when they are done, so you can follow up with this when all is production ready:)

@iljamak
Copy link
Collaborator

iljamak commented May 19, 2024

@filip-michalsky
I runned the test locally and tested functionality of all tools (StripePaymentLink, CalendlyLink, Gmail, Database Search). I've also reviewed the process of gmail app password generation, which describes it the way i was doing it.
At the moment of writing previous comment, i didnt runned it inside docker container, however i've tested it and it runs correctly.
I have verified all steps from A to E for the last commit

@filip-michalsky
Copy link
Owner

awesome thank you @iljamak

@chemik-bit pls let us know when this is ready for prime time!

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