Dev #352
Merged
Dev #352
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…lity score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis
* Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai>
* Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to main (#300) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai>
…p manager account in the daily analysis for now (#308)
* Dev (#303) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to main (#300) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Add redirect domain --------- Co-authored-by: rjambrecic <32619626+rjambrecic@users.noreply.github.com> * 305 metrics cannot be requested for a manager account (#307) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to main (#300) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Add redirect domain * Quick fix for metrics cannot be requested for a manager account - skip manager account in the daily analysis for now --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai>
…315) * Add connect_timeout parameter to database url * Add display path1 and path2 fields to Ads
… the permission before the daily analysis (#317)
* Remove litellm variables * Use function to check missing bash variable * Optimize ssh commands
…le Ads permission (#323)
* Fix typo in sweden * Add separate secret variables for azure openai creds
* WIP * WIP: add email template * WIP: add email template * WIP: add email template * Add email HTML templates * Add proposed actions to email template
* Implement add_geo_targeting_to_campaign endpoint * Integrate create_geo_targeting_for_campaign to the Google ads team * Update tests for geo targeting * Fix execute_query exception message when gads authentication error happens * Update prompts for gads geo targeting
* Update html template * Fix broken html template * Integrate email template for Daily analysis * Execute daily analysis for everyone when send_only_to_emails parameter is None --------- Co-authored-by: Harish Mohan Raj <harish@airt.ai>
* Dev (#338) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to main (#300) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Add redirect domain * Quick fix for metrics cannot be requested for a manager account - skip manager account in the daily analysis for now (#308) * Merge main to dev (#311) * Dev (#303) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to main (#300) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Add redirect domain --------- Co-authored-by: rjambrecic <32619626+rjambrecic@users.noreply.github.com> * 305 metrics cannot be requested for a manager account (#307) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Merge to main (#300) * Add currency next to the cost per click --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Add redirect domain * Quick fix for metrics cannot be requested for a manager account - skip manager account in the daily analysis for now --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> * Allow only dev to merge into main (#313) * 247 use display path1 and path2 attributes when creating-updating ad (#315) * Add connect_timeout parameter to database url * Add display path1 and path2 fields to Ads * Fix daily analysis prompt (#316) * Create chat with login url as a first message if the user didn't give the permission before the daily analysis (#317) * Send chat exception to frontend (#319) * Cleanup env variables (#318) * Remove litellm variables * Use function to check missing bash variable * Optimize ssh commands * Fix HTML message displayed in the frontend (#320) * Send daily analysis only to people who have already grant us the Google Ads permission (#323) * Add separate secret variables for azure openai credentials (#324) * Fix typo in sweden * Add separate secret variables for azure openai creds * Remove unnecessary variable * Regenerate gads credentials if they are not valid (#327) * Delete users google ads credentials only when RefreshError happens (#330) * Create email html template (#333) * WIP * WIP: add email template * WIP: add email template * WIP: add email template * Add email HTML templates * Add proposed actions to email template * 167 add geo targeting criteria country city (#334) * Implement add_geo_targeting_to_campaign endpoint * Integrate create_geo_targeting_for_campaign to the Google ads team * Update tests for geo targeting * Fix execute_query exception message when gads authentication error happens * Update prompts for gads geo targeting * Remove unnecessary env variable (#336) * 332 integrate html email template (#337) * Update html template * Fix broken html template * Integrate email template for Daily analysis * Execute daily analysis for everyone when send_only_to_emails parameter is None --------- Co-authored-by: Harish Mohan Raj <harish@airt.ai> * Add templates directory inside the Dockerfile (#339) --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> Co-authored-by: Harish Mohan Raj <harish@airt.ai> * Handle azure openai 400 error in teams conversation --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> Co-authored-by: Harish Mohan Raj <harish@airt.ai>
… targeted (#346) * Update python libraries versions * Fix geo targeting endpoint * Update prompts for geo targeting
* Run tests in CI * Add dummy env vars * Do editable install * Add prisma generate * Add dummy client secrets * Debug * Fix typo * Run postgres container --------- Co-authored-by: rjambrecic <32619626+rjambrecic@users.noreply.github.com>
* Update prompts with instructions for the keyword insertion
* Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- * Merge to main (#300) * Add currency next to the cost per click --------- --------- * Add redirect domain * Quick fix for metrics cannot be requested for a manager account - skip manager account in the daily analysis for now (#308) * Merge main to dev (#311) * Dev (#303) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- * Merge to main (#300) * Add currency next to the cost per click --------- --------- * Add redirect domain --------- * 305 metrics cannot be requested for a manager account (#307) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- * Merge to main (#300) * Add currency next to the cost per click --------- --------- * Add redirect domain * Quick fix for metrics cannot be requested for a manager account - skip manager account in the daily analysis for now --------- * Allow only dev to merge into main (#313) * 247 use display path1 and path2 attributes when creating-updating ad (#315) * Add connect_timeout parameter to database url * Add display path1 and path2 fields to Ads * Fix daily analysis prompt (#316) * Create chat with login url as a first message if the user didn't give the permission before the daily analysis (#317) * Send chat exception to frontend (#319) * Cleanup env variables (#318) * Remove litellm variables * Use function to check missing bash variable * Optimize ssh commands * Fix HTML message displayed in the frontend (#320) * Send daily analysis only to people who have already grant us the Google Ads permission (#323) * Add separate secret variables for azure openai credentials (#324) * Fix typo in sweden * Add separate secret variables for azure openai creds * Remove unnecessary variable * Regenerate gads credentials if they are not valid (#327) * Delete users google ads credentials only when RefreshError happens (#330) * Create email html template (#333) * WIP * WIP: add email template * WIP: add email template * WIP: add email template * Add email HTML templates * Add proposed actions to email template * 167 add geo targeting criteria country city (#334) * Implement add_geo_targeting_to_campaign endpoint * Integrate create_geo_targeting_for_campaign to the Google ads team * Update tests for geo targeting * Fix execute_query exception message when gads authentication error happens * Update prompts for gads geo targeting * Remove unnecessary env variable (#336) * 332 integrate html email template (#337) * Update html template * Fix broken html template * Integrate email template for Daily analysis * Execute daily analysis for everyone when send_only_to_emails parameter is None --------- * Add templates directory inside the Dockerfile (#339) --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> Co-authored-by: Harish Mohan Raj <harish@airt.ai>
* Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- * Merge to main (#300) * Add currency next to the cost per click --------- --------- * Add redirect domain * Quick fix for metrics cannot be requested for a manager account - skip manager account in the daily analysis for now (#308) * Merge main to dev (#311) * Dev (#303) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- * Merge to main (#300) * Add currency next to the cost per click --------- --------- * Add redirect domain --------- * 305 metrics cannot be requested for a manager account (#307) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis * Add currency next to the cost per click (#298) * Add currency next to the cost per click --------- * Merge to dev (#301) * Dev (#294) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * Dev (#296) * Add staging docker build and deployment (#289) * Deploy in dev to staging * Experiment with if else * Fix CI secrets * Cleanup unnecessary stuff * 286 add an end point for triggering daily analysis emails and add quality score to daily analysis (#295) * Implement daily-analysis endpoint * Add quality score to daily analysis --------- * Merge to main (#300) * Add currency next to the cost per click --------- --------- * Add redirect domain * Quick fix for metrics cannot be requested for a manager account - skip manager account in the daily analysis for now --------- * Allow only dev to merge into main (#313) * 247 use display path1 and path2 attributes when creating-updating ad (#315) * Add connect_timeout parameter to database url * Add display path1 and path2 fields to Ads * Fix daily analysis prompt (#316) * Create chat with login url as a first message if the user didn't give the permission before the daily analysis (#317) * Send chat exception to frontend (#319) * Cleanup env variables (#318) * Remove litellm variables * Use function to check missing bash variable * Optimize ssh commands * Fix HTML message displayed in the frontend (#320) * Send daily analysis only to people who have already grant us the Google Ads permission (#323) * Add separate secret variables for azure openai credentials (#324) * Fix typo in sweden * Add separate secret variables for azure openai creds * Remove unnecessary variable * Regenerate gads credentials if they are not valid (#327) * Delete users google ads credentials only when RefreshError happens (#330) * Create email html template (#333) * WIP * WIP: add email template * WIP: add email template * WIP: add email template * Add email HTML templates * Add proposed actions to email template * 167 add geo targeting criteria country city (#334) * Implement add_geo_targeting_to_campaign endpoint * Integrate create_geo_targeting_for_campaign to the Google ads team * Update tests for geo targeting * Fix execute_query exception message when gads authentication error happens * Update prompts for gads geo targeting * Remove unnecessary env variable (#336) * 332 integrate html email template (#337) * Update html template * Fix broken html template * Integrate email template for Daily analysis * Execute daily analysis for everyone when send_only_to_emails parameter is None --------- * Add templates directory inside the Dockerfile (#339) --------- Co-authored-by: Kumaran Rajendhiran <kumaran@airt.ai> Co-authored-by: Harish Mohan Raj <harish@airt.ai>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.