Skip to content
This repository was archived by the owner on Apr 3, 2025. It is now read-only.

Commit c56fdf6

Browse files
authored
Merge pull request #21 from google/rename_add_a
Renames megalist to megalista
2 parents 346d063 + fefc8c0 commit c56fdf6

File tree

64 files changed

+31
-27
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

64 files changed

+31
-27
lines changed

.github/workflows/python-app.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
- name: Install dependencies
2424
run: |
2525
python -m pip install --upgrade pip
26-
pip install -r megalist_dataflow/requirements.txt
26+
pip install -r megalista_dataflow/requirements.txt
2727
- name: Run tests
2828
run: |
2929
./run_tests.sh

README.md

+5-5

cloud_config/buyers.json

-2
This file was deleted.

cloud_config/scheduler_sample.json

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
{
2-
"jobName": "megalist_daily",
2+
"jobName": "megalista_daily",
33
"parameters": {
44
"developer_token": "Google Ads Developer Token",
55
"client_id": "GCP OAuth Client id",
@@ -13,7 +13,7 @@
1313
"appsflyer_dev_key": "Apps flyer dev key"
1414
},
1515
"environment": {
16-
"tempLocation": "gs://megalist-data/temp",
16+
"tempLocation": "gs://megalista-data/temp",
1717
"zone": "us-central1-f"
1818
}
1919
}

generate_megalist_token.sh renamed to generate_megalista_token.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,5 +18,5 @@ if [ $# != 2 ]; then
1818
exit 1
1919
fi
2020

21-
pip3 install --user -q -r megalist_dataflow/requirements.txt
22-
python3 cloud_config/generate_megalist_token.py --client_id $1 --client_secret $2
21+
pip3 install --user -q -r megalista_dataflow/requirements.txt
22+
python3 cloud_config/generate_megalista_token.py --client_id $1 --client_secret $2
File renamed without changes.

megalist_dataflow/megalist_metadata renamed to megalista_dataflow/megalista_metadata

+5-4
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
{
2-
"name": "Megalist",
3-
"description": "Buyers audience generator and uploader",
2+
"name": "Megalista",
3+
"description": "Data integration automation solution",
44
"parameters": [
55
{
66
"name": "developer_token",
@@ -54,8 +54,9 @@
5454
},
5555
{
5656
"name": "bq_ops_dataset",
57-
"label": "Auxiliary BigQuery dataset used for Megalista operations",
58-
"help_text": "Auxiliary BigQuery dataset used for Megalista operations"
57+
"label": "Auxiliary bigquery dataset used for Megalista operations",
58+
"help_text": "Auxiliary bigquery dataset used for Megalista operations",
59+
"is_optional": "true"
5960
},
6061
{
6162
"name": "appsflyer_dev_key",

megalist_dataflow/setup.py renamed to megalista_dataflow/setup.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515
import setuptools
1616

1717
setuptools.setup(
18-
name='megalist_dataflow',
18+
name='megalista_dataflow',
1919
version='4.1',
2020
author='Google',
2121
author_email='[email protected]',

megalist_dataflow/uploaders/google_ads/customer_match/abstract_uploader.py renamed to megalista_dataflow/uploaders/google_ads/customer_match/abstract_uploader.py

+8-3
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,8 @@
1313
# limitations under the License.
1414

1515
import logging
16-
from typing import Dict, Any, List
16+
from typing import Dict, Any, List, Optional
17+
from apache_beam.options.value_provider import StaticValueProvider
1718

1819
import apache_beam as beam
1920
from uploaders import utils
@@ -28,7 +29,7 @@
2829

2930
class GoogleAdsCustomerMatchAbstractUploaderDoFn(beam.DoFn):
3031

31-
def __init__(self, oauth_credentials: OAuthCredentials, developer_token: str):
32+
def __init__(self, oauth_credentials: OAuthCredentials, developer_token: StaticValueProvider):
3233
super().__init__()
3334
self.oauth_credentials = oauth_credentials
3435
self.developer_token = developer_token
@@ -83,11 +84,15 @@ def _do_create_list_if_it_does_not_exist(self,
8384
else:
8485
logging.getLogger(_DEFAULT_LOGGER).info('List %s found with resource name: %s',
8586
list_name, resource_name)
86-
return resource_name
87+
return str(resource_name)
8788

89+
<<<<<<< HEAD
8890
def _get_user_list_resource_name(self, customer_id: str, list_name: str):
8991
ads_client = utils.get_ads_client(self.oauth_credentials, self.developer_token.get(), customer_id)
9092

93+
=======
94+
def _get_user_list_resource_name(self, customer_id: str, list_name: str) -> Optional[str]:
95+
>>>>>>> Changes to adhere to MyPy
9196
resource_name = None
9297
service = self._get_ads_service(customer_id)
9398

run_cloud.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -21,4 +21,4 @@ fi
2121

2222
gcloud config set project $1
2323
token=$(gcloud auth application-default print-access-token)
24-
curl -H "Authorization: Bearer $token" -H "Content-Type:application/json" "https://dataflow.googleapis.com/v1b3/projects/$1/locations/$3/templates:launch?gcsPath=gs://$2/templates/megalist" --data-binary "@cloud_config/scheduler.json"
24+
curl -H "Authorization: Bearer $token" -H "Content-Type:application/json" "https://dataflow.googleapis.com/v1b3/projects/$1/locations/$3/templates:launch?gcsPath=gs://$2/templates/megalista" --data-binary "@cloud_config/scheduler.json"

run_tests.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -13,5 +13,5 @@
1313
# See the License for the specific language governing permissions and
1414
# limitations under the License.
1515

16-
python3 -m mypy megalist_dataflow
17-
python3 -m pytest -vv --cov=megalist_dataflow -W ignore::DeprecationWarning
16+
python3 -m mypy megalista_dataflow
17+
python3 -m pytest -vv --cov=megalista_dataflow -W ignore::DeprecationWarning

terraform/main.tf

+1-1
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ resource "null_resource" "only_one_configuration_provided" {
1818
locals {
1919
scheduler_body = <<EOF
2020
{
21-
"jobName": "megalist_daily",
21+
"jobName": "megalista_daily",
2222
"parameters": {
2323
"developer_token": "${var.developer_token}",
2424
"client_id": "${var.client_id}",

terraform/scripts/deploy_cloud.sh

+3-3
Original file line numberDiff line numberDiff line change
@@ -19,13 +19,13 @@ if [ $# != 3 ]; then
1919
exit 1
2020
fi
2121

22-
echo "Move to megalist_dataflow folder"
23-
cd megalist_dataflow
22+
echo "Move to megalista_dataflow folder"
23+
cd megalista_dataflow
2424
echo "Configuration GCP project in gcloud"
2525
gcloud config set project "$1"
2626
echo "Build Dataflow metadata"
2727
python3 -m pip install --user -q -r requirements.txt
2828
python3 -m main --runner DataflowRunner --project "$1" --gcp_project_id "$1" --temp_location "gs://$2/tmp/" --region "$3" --setup_file ./setup.py --template_location "gs://$2/templates/megalista" --num_workers 1 --autoscaling_algorithm=NONE
2929
echo "Copy megalista_medata to bucket $2"
30-
gsutil cp megalist_metadata "gs://$2/templates/megalista_metadata"
30+
gsutil cp megalista_metadata "gs://$2/templates/megalista_metadata"
3131
cd ..

0 commit comments

Comments
 (0)