-
Notifications
You must be signed in to change notification settings - Fork 0
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
Dev #348
Closed
Closed
Dev #348
Commits on Feb 16, 2024
-
* 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>
Configuration menu - View commit details
-
Copy full SHA for 9e25e95 - Browse repository at this point
Copy the full SHA 9e25e95View commit details
Commits on Feb 20, 2024
-
Configuration menu - View commit details
-
Copy full SHA for 1d0a492 - Browse repository at this point
Copy the full SHA 1d0a492View commit details
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.