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

Feature new vertical zartantv #499

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

Conversation

mdwallick
Copy link
Contributor

@mdwallick mdwallick commented Feb 9, 2022

This is a new vertical, based on streamingservice, using the now OOTB device authorization grant. This removes all of the custom code used in the streamingservice vertical.


Resolves #499 463

@mdwallick mdwallick added the type: feature New feature or request label Feb 9, 2022
@mdwallick mdwallick marked this pull request as draft February 9, 2022 13:16
@mdwallick mdwallick linked an issue Feb 9, 2022 that may be closed by this pull request
@noinarisak noinarisak self-requested a review February 9, 2022 15:27
@noinarisak
Copy link
Contributor

@mdwallick This our fault, since there is no .gitignore preventing .tfplan files from getting committed. Please remove terraform/zartantv/okta.setup.tfplan since that should not be git.

I plan to merge PR #468 that will fix that. You will have to changes things up though, since I'm separating the UDP related HCL scripts from LocalHost HCL scripts.

@sonarcloud
Copy link

sonarcloud bot commented Mar 22, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 2 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mdwallick mdwallick changed the title [DRAFT] Feature zartantv iss463 [DRAFT] Feature new vertical zartantv Mar 25, 2022
@mdwallick mdwallick changed the title [DRAFT] Feature new vertical zartantv Feature new vertical zartantv Mar 25, 2022
@mdwallick mdwallick marked this pull request as ready for review March 25, 2022 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE] ZartanTV - new veritcal - Use Device Authorization Grant
2 participants