Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Signal pixel on new config merged #144

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

cmonfortep
Copy link
Collaborator

Related to: https://app.asana.com/0/1207619243206445/1208965192384904/f

Description:
Updates workflows so they trigger a pixel on any platform config merge.
That can be consumed from grafana and they can be displayed as labels where needed.

@@ -29,23 +29,4 @@ jobs:
if: github.event.action == 'opened'
with:
repo-token: ${{ secrets.GITHUB_TOKEN }}
message: 'Your PR is hosted at https://staticcdn.duckduckgo.com/remotemessaging/config/staging/android-config.json'
- name: Extract Version from JSON
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing to see here. Just a test I've done to validate this works using android staging workflow.

Copy link
Contributor

@ayoy ayoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 👍

- name: Trigger Network Request
id: trigger_request
run: |
RESPONSE=$(curl -s -o response.txt -w "%{http_code}" "https://improving.duckduckgo.com/t/m_new_rmf_config_ios?version=${VERSION}")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To limit curl output even more you can use the following:

Suggested change
RESPONSE=$(curl -s -o response.txt -w "%{http_code}" "https://improving.duckduckgo.com/t/m_new_rmf_config_ios?version=${VERSION}")
RESPONSE=$(curl -fLSs -o /dev/null -w "%{http_code}" "https://improving.duckduckgo.com/t/m_new_rmf_config_ios?version=${VERSION}")

Copy link
Contributor

@Jonathan-L Jonathan-L left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Collaborator

@amddg44 amddg44 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants