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

Removing a / thats breaking the AWS console Load Balancer URL in the UI #915

Merged

Conversation

pavangudiwada
Copy link
Contributor

@pavangudiwada pavangudiwada commented Aug 21, 2023

Problem

Solves the issue #914

Solution

Remove the / in the URL

Changes Made

  • Removed a / after # in the URL

How to Test

  • Build and run the current Develop branch
  • Go to Inventory -> All resources -> Filter -> Cloud service -> is -> ELB
  • Click one of the Load Balancers and click the link beside the Load Balancer's name

Screenshots

image

Checklist

  • Code follows the contributing guidelines
  • Changes have been thoroughly tested
  • Documentation has been updated, if necessary
  • Any dependencies have been added to the project, if necessary

Copy link
Collaborator

@mlabouardy mlabouardy left a comment

Choose a reason for hiding this comment

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

Thanks for the fix @pavangudiwada

@mlabouardy mlabouardy added the aws label Aug 21, 2023
@mlabouardy mlabouardy modified the milestones: v3.1.0, v3.1.1 Aug 21, 2023
@mlabouardy mlabouardy merged commit f65b32e into tailwarden:develop Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants