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

Remove Analytics scope from Google Api Client #7407

Merged
merged 1 commit into from
Jan 16, 2025

Conversation

scruti
Copy link
Collaborator

@scruti scruti commented Jan 14, 2025

This scope was introduced to access/store the vacancies view counters data from Google Analytics. Was done with this PR:

Since then, those counter db fields have been removed, the Google Analytics gem & client also have been removed.

Our current project only uses Google Api client for Google Drive and Indexing.

All the statistical data gets queried directly through Google Bigquery client. Called from the Publishers::VacancyStats service.

This scope was introduced to access/store the vacancies view counters
data from Google Analytics. Was done with this PR:
- #547

Since then, those counter db fields have been removed, the Google
Analytics gem & client also have been removed.

Our current project only uses Google Api client for Google Drive and
Indexing.

All the statistical data gets queried directly through Google Bigquery
client. Called from the Publishers::VacancyStats service.
@scruti scruti added the deploy label Jan 14, 2025
@scruti scruti marked this pull request as draft January 14, 2025 11:33
Copy link

github-actions bot commented Jan 14, 2025

Review app https://teaching-vacancies-review-pr-7407.test.teacherservices.cloud was successfully deleted

@scruti scruti marked this pull request as ready for review January 15, 2025 16:22
@scruti scruti merged commit c4b96a4 into main Jan 16, 2025
13 checks passed
@scruti scruti deleted the remove-analytics-scope-from-google-api-client branch January 16, 2025 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants