Skip to content

Update service discovery feature descriptions in documentation - #4168

Open
nassergkhaled wants to merge 1 commit into
spring-cloud:mainfrom
nassergkhaled:main
Open

nassergkhaled wants to merge 1 commit into
spring-cloud:mainfrom
nassergkhaled:main

Conversation

@nassergkhaled

Copy link
Copy Markdown
Contributor

Updated the Features section in readme.adoc and the Spring Cloud Netflix features section in segan-index.adoc to better reflect the actual features provided by Spring Cloud Netflix. Replaced "Service Discovery" with "Service Registration" and "Service Discovery" to more accurately describe the functionality. Also added a new section on "Sample" to provide more context for the features.

@pivotal-cla

Copy link
Copy Markdown

@nasserkh12 Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@ankit-gautam23 ankit-gautam23 left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@OlgaMaciaszek OlgaMaciaszek self-assigned this Sep 26, 2023
@OlgaMaciaszek OlgaMaciaszek added this to the 3.1.8 milestone Sep 26, 2023

@OlgaMaciaszek OlgaMaciaszek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks @nassergkhaled. LGTM.

@OlgaMaciaszek

Copy link
Copy Markdown
Collaborator

Thanks, @nassergkhaled. Please sign the CLA.

@spencergibb spencergibb modified the milestones: 3.1.8, 4.1.1 Feb 7, 2024
@spencergibb spencergibb removed this from the 4.1.1 milestone Apr 3, 2024
@ryanjbaxter ryanjbaxter removed this from 2023.0.6 Jun 27, 2025

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

7 participants