docs icon indicating copy to clipboard operation
docs copied to clipboard

Update using-ldap.md

Open adamrr724 opened this issue 2 years ago â€ĸ 2 comments
trafficstars

Changing enable LDAP Sync header to ## so that it shows up in quick links along the right side of the article.

Why: Attempting to add this to the quick links on the right side of the article by updating the header type.

Closes: No issue created.

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

  • [ ] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline.

    • For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.
  • [ ] For content changes, I have completed the self-review checklist.

adamrr724 avatar Jun 09 '23 21:06 adamrr724

Automatically generated comment â„šī¸

This comment is automatically generated and will be overwritten every time changes are committed to this branch.

The table contains an overview of files in the content directory that have been changed in this pull request. It's provided to make it easy to review your changes on the staging site. Please note that changes to the data directory will not show up in this table.


Content directory changes

You may find it useful to copy this table into the pull request summary. There you can edit it to share links to important articles or changes and to give a high-level overview of how the changes in your pull request support the overall goals of the pull request.

Source Preview Production What Changed
admin/identity-and-access-management/using-ldap-for-enterprise-iam/using-ldap.md ghes@ 3.11 3.10 3.9 3.8
ghes@ 3.11 3.10 3.9 3.8

fpt: Free, Pro, Team ghec: GitHub Enterprise Cloud ghes: GitHub Enterprise Server

github-actions[bot] avatar Jun 09 '23 21:06 github-actions[bot]

👋 @adamrr724 Thanks so much for opening a PR! I'll get this triaged for review :zap:

cmwilson21 avatar Jun 12 '23 13:06 cmwilson21

Hi @adamrr724! 👋 Just wanted to check if you saw @lecoursen's last comment regarding the H2s. If there's some context I'm missing, and this PR should just be closed out, let me know! 💛

nguyenalex836 avatar Jan 11 '24 21:01 nguyenalex836

@lecoursen Hi Laura! Would you be up for giving this PR a final review now that @adamrr724 has updated to H2s?

nguyenalex836 avatar Jan 12 '24 16:01 nguyenalex836

Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues :zap:

github-actions[bot] avatar Jan 12 '24 18:01 github-actions[bot]