Skip to content
This repository has been archived by the owner on Apr 19, 2023. It is now read-only.

added resource level tagging #205

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

added resource level tagging #205

wants to merge 7 commits into from

Conversation

smp
Copy link
Contributor

@smp smp commented Feb 18, 2021

Issue #, if available:
added resource level tagging for mediaconvert jobs, cloudfront distros, and IVS channels. Part of #152

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@smp smp requested a review from wizage February 18, 2021 05:42
@wizage
Copy link
Contributor

wizage commented Feb 18, 2021

Can you rebase on top of master? Changes were made to one of the templates.

nathanagez and others added 6 commits February 22, 2021 00:04
* feat(jest): Add jest, eslint plugin and update package.json

* feat(jest): Add sample files

* refactor(docs): Add tests section to CONTRIBUTING.md

* feat(gh-action): Add a GH Action for Jest
@smp
Copy link
Contributor Author

smp commented Feb 27, 2021

Done.

armenr added a commit to armenr/amplify-video that referenced this pull request May 6, 2021
wizage pushed a commit that referenced this pull request Jun 4, 2021
* fix(props): Extend code to support individual props files & support multiple backend envs

* fix(multiEnv): committing WIP - half-working test script for generating new props from existing props

* fix(video-staging): Added logic for generating unique props from existing video resource

* fix(build): Added logic to build props from existing resource when build is invoked

* fix(tokenGen): Fix CloudFront Token Gen Lambda (as per pull #247)

* fix(tokenGen): Added env interpolation to tokenGen GQL schema (as per pr #219)

* fix(gqlSchema): Capitalize GQL models for Admin-UI compliance/friendliness

* fix(tagging): Added resource-level tagging in accordance with pr #205
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants