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

docs: update links to the calling-compilecomponents section #55055

Closed
wants to merge 2 commits into from

Conversation

amayer42
Copy link
Contributor

PR Type

What kind of change does this PR introduce?

  • Documentation content changes

What is the current behavior?

Clicking on links to the Calling compileComponents() section do not apply the correct fragment. Instead of "calling-compilecomponents" they set the fragment to "compile-components".

Issue Number: N/A

What is the new behavior?

I have updated links to the section to use "calling-compilecomponents" as the fragment instead of "compile-components".

Does this PR introduce a breaking change?

  • Yes
  • No

@pullapprove pullapprove bot requested a review from alxhub March 26, 2024 18:25
@angular-robot angular-robot bot added the area: docs Related to the documentation label Mar 26, 2024
@ngbot ngbot bot added this to the Backlog milestone Mar 26, 2024
Copy link

github-actions bot commented Mar 26, 2024

Deployed adev-preview for c7e50e0 to: https://ng-dev-previews-fw--pr-angular-angular-55055-adev-prev-wzsqrc62.web.app

Note: As new commits are pushed to this pull request, this link is updated after the preview is rebuilt.

Copy link
Member

@JeanMeche JeanMeche left a comment

Choose a reason for hiding this comment

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

LGTM

@JeanMeche JeanMeche added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release and removed action: merge The PR is ready for merge by the caretaker labels Mar 26, 2024
@bencodezen bencodezen added the action: merge The PR is ready for merge by the caretaker label Mar 27, 2024
@atscott atscott added target: minor This PR is targeted for the next minor release action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews and removed target: patch This PR is targeted for the next patch release action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Mar 29, 2024
@atscott
Copy link
Contributor

atscott commented Mar 29, 2024

@amayer42 could you please rebase this PR? Looks like there are conflicts with other things that got merged

@amayer42
Copy link
Contributor Author

@atscott hoping that I did it right, never really used rebase before. The commits look right to me now though, so fingers crossed.

@amayer42
Copy link
Contributor Author

amayer42 commented Apr 4, 2024

@atscott @JeanMeche , is there anything else that I need to do on this? Or does this just need @alxhub 's approval at this point? I just want to make sure that I'm not missing something.

@atscott atscott added action: merge The PR is ready for merge by the caretaker and removed action: cleanup The PR is in need of cleanup, either due to needing a rebase or in response to comments from reviews labels Apr 5, 2024
@atscott atscott added the target: patch This PR is targeted for the next patch release label Apr 8, 2024
@atscott
Copy link
Contributor

atscott commented Apr 8, 2024

This PR was merged into the repository by commit 2eb83c6.

@atscott atscott closed this in 76d00b2 Apr 8, 2024
@amayer42 amayer42 deleted the patch-1 branch April 9, 2024 14:20
iteriani pushed a commit to iteriani/angular that referenced this pull request Apr 11, 2024
iteriani pushed a commit to iteriani/angular that referenced this pull request Apr 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker adev: preview area: docs Related to the documentation target: patch This PR is targeted for the next patch release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants