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

website/integrations: argocd: add missing url in ArgoCD configuration #7404

Merged
merged 1 commit into from
Nov 1, 2023

Conversation

gc4g40u6
Copy link
Contributor

@gc4g40u6 gc4g40u6 commented Nov 1, 2023

Details

Should also add the argocd url in the argocd-cm

Signed-off-by: gc4g40u6 <137039372+gc4g40u6@users.noreply.github.com>
@gc4g40u6 gc4g40u6 requested a review from a team as a code owner November 1, 2023 10:12
Copy link

netlify bot commented Nov 1, 2023

Deploy Preview for authentik-storybook canceled.

Name Link
🔨 Latest commit 925b97d
🔍 Latest deploy log https://app.netlify.com/sites/authentik-storybook/deploys/6542248b1c72b500084a753d

Copy link

netlify bot commented Nov 1, 2023

Deploy Preview for authentik ready!

Name Link
🔨 Latest commit 925b97d
🔍 Latest deploy log https://app.netlify.com/sites/authentik/deploys/6542248b1df5a20008b1a35c
😎 Deploy Preview https://deploy-preview-7404--authentik.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 95 (🔴 down 1 from production)
Accessibility: 90 (no change from production)
Best Practices: 100 (no change from production)
SEO: 80 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

@rissson rissson changed the title Update index.md website/integrations: argocd: add missing url in ArgoCD configuration Nov 1, 2023
@rissson rissson enabled auto-merge (squash) November 1, 2023 11:41
Copy link

codecov bot commented Nov 1, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (345022f) 92.17% compared to head (925b97d) 92.05%.
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7404      +/-   ##
==========================================
- Coverage   92.17%   92.05%   -0.13%     
==========================================
  Files         587      587              
  Lines       28895    28895              
==========================================
- Hits        26634    26598      -36     
- Misses       2261     2297      +36     
Flag Coverage Δ
e2e 48.60% <ø> (-0.73%) ⬇️
integration 26.01% <ø> (ø)
unit 89.59% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@rissson rissson merged commit a10392e into goauthentik:main Nov 1, 2023
64 checks passed
kensternberg-authentik added a commit that referenced this pull request Nov 1, 2023
* main:
  website/integrations: argocd: add missing url in ArgoCD configuration (#7404)
  core: bump sentry-sdk from 1.32.0 to 1.33.1 (#7397)
  core: bump webauthn from 1.11.0 to 1.11.1 (#7399)
  core: bump github.com/redis/go-redis/v9 from 9.2.1 to 9.3.0 (#7396)
  core: bump twisted from 23.8.0 to 23.10.0 (#7398)
  web: bump the sentry group in /web with 2 updates (#7401)
  web: bump pyright from 1.1.333 to 1.1.334 in /web (#7402)
  web: bump rollup from 4.1.5 to 4.2.0 in /web (#7403)
  core: bump pytest-django from 4.5.2 to 4.6.0 (#7387)
  web: bump the eslint group in /tests/wdio with 2 updates (#7388)
  web: bump the sentry group in /web with 2 updates (#7366)
  web: bump the eslint group in /web with 2 updates (#7389)
  web: bump core-js from 3.33.1 to 3.33.2 in /web (#7390)
  stages/email: fix duplicate querystring encoding (#7386)
  web/admin: fix html error on oauth2 provider page (#7384)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants