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

secscan: Correct links (PROJQUAY-2164) #1552

Merged
merged 1 commit into from
Dec 2, 2022

Conversation

crozzy
Copy link
Contributor

@crozzy crozzy commented Oct 7, 2022

Use first link in the "array", don't display link icon when no links exist.

Signed-off-by: crozzy joseph.crosland@gmail.com

Use first link in the "array", don't display link icon when
no links exist.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
@crozzy
Copy link
Contributor Author

crozzy commented Oct 7, 2022

Figured I'd look at this while I have the whole dev env set up

@crozzy crozzy merged commit 84786b9 into quay:master Dec 2, 2022
HammerMeetNail pushed a commit to HammerMeetNail/quay that referenced this pull request Dec 7, 2022
Use first link in the "array", don't display link icon when
no links exist.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
HammerMeetNail added a commit that referenced this pull request Dec 7, 2022
* Util: Adding git pre-commit checks (PROJQUAY-4658) (#1585)

* Util: Adding git pre-commit checks (PROJQUAY-0000)

* updating getting started markdown

* ui: Add script for adobe analytics for quay.io (PROJQUAY-4582) (#1648)

Analytics will be added only when building for quay.io

* secscan: Correct links (PROJQUAY-2164) (#1552)

Use first link in the "array", don't display link icon when
no links exist.

Signed-off-by: crozzy <joseph.crosland@gmail.com>

* ui: Show UI toggle on quay.io only to redhat users (PROJQUAY-4804) (#1653)

ui: Show UI toggle on quay.io only to redhat users (PROJQUAY-4804)

New UI preview on quay.io is being released as a preview only
to RedHat users before being GA

* Update nginx config (#1639)

Signed-off-by: Dave O'Connor <doconnor@redhat.com>

* ui: Add script for adobe analytics for quay.io in angular UI (PROJQUAY-4582) (#1654)

This will be used for usage analytics on quay.io

* Add cookie consent for quay.io (#1657)

* ui: Remove FEATURE_UI_V2 from analytics scripts (PROJQUAY-4582) (#1658)

* ui: Remove FEATURE_ from feature_set in base template (PROJQUAY-4582)

the variable `feature_set` has features with the `FEATURE_` prefix
removed. Fix the jinja check for this

* remove feature_ui_v2

Signed-off-by: crozzy <joseph.crosland@gmail.com>
Signed-off-by: Dave O'Connor <doconnor@redhat.com>
Co-authored-by: Sunandadadi <Sunandadadi@users.noreply.github.com>
Co-authored-by: Syed Ahmed <syed@apache.org>
Co-authored-by: Joseph Crosland <joseph.crosland@gmail.com>
HammerMeetNail added a commit to openshift-cherrypick-robot/quay that referenced this pull request Dec 12, 2022
* Util: Adding git pre-commit checks (PROJQUAY-4658) (quay#1585)

* Util: Adding git pre-commit checks (PROJQUAY-0000)

* updating getting started markdown

* ui: Add script for adobe analytics for quay.io (PROJQUAY-4582) (quay#1648)

Analytics will be added only when building for quay.io

* secscan: Correct links (PROJQUAY-2164) (quay#1552)

Use first link in the "array", don't display link icon when
no links exist.

Signed-off-by: crozzy <joseph.crosland@gmail.com>

* ui: Show UI toggle on quay.io only to redhat users (PROJQUAY-4804) (quay#1653)

ui: Show UI toggle on quay.io only to redhat users (PROJQUAY-4804)

New UI preview on quay.io is being released as a preview only
to RedHat users before being GA

* Update nginx config (quay#1639)

Signed-off-by: Dave O'Connor <doconnor@redhat.com>

* ui: Add script for adobe analytics for quay.io in angular UI (PROJQUAY-4582) (quay#1654)

This will be used for usage analytics on quay.io

* Add cookie consent for quay.io (quay#1657)

* ui: Remove FEATURE_UI_V2 from analytics scripts (PROJQUAY-4582) (quay#1658)

* ui: Remove FEATURE_ from feature_set in base template (PROJQUAY-4582)

the variable `feature_set` has features with the `FEATURE_` prefix
removed. Fix the jinja check for this

* remove feature_ui_v2

Signed-off-by: crozzy <joseph.crosland@gmail.com>
Signed-off-by: Dave O'Connor <doconnor@redhat.com>
Co-authored-by: Sunandadadi <Sunandadadi@users.noreply.github.com>
Co-authored-by: Syed Ahmed <syed@apache.org>
Co-authored-by: Joseph Crosland <joseph.crosland@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants