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

setup.xml: replace the broken URL #3551

Closed
wants to merge 1 commit into from
Closed

Conversation

mmalferov
Copy link
Member

The Microsoft Knowledge Base does not contain the "Best Practices and Recommendations for authors of Discretionary Access Control Service Lists" page. And I couldn't find a copy of the material on the MKB website. I have replaced the link to the most reputable, in my humble opinion, site, which still keeps a copy of the original article

The Microsoft Knowledge Base does not contain the "Best Practices and Recommendations for authors of Discretionary Access Control Service Lists" page. And I couldn't find a copy of the material on the MKB website. I have replaced the link to the most reputable, in my humble opinion, site, which still keeps a copy of the original article
@cmb69
Copy link
Member

cmb69 commented Jul 11, 2024

Hmm, that article is apparently very old anyway. Maybe @macintoshplus has an idea where to link to instead?

@macintoshplus
Copy link
Contributor

I haven't found the new documentation page. This link is a mirror of old Microsoft documentation.

This process of managing service rights is not efficient. I prepare a new version of Win32Service with a new method to replace this tool.

@Girgias Girgias added the QA Quality Assurance label Jul 26, 2024
@macintoshplus macintoshplus self-requested a review October 15, 2024 06:59
@macintoshplus
Copy link
Contributor

The version 1.1.0 with the replacement method is in beta release.

I prefer to keep a valid link for old usage and add the new rights management process available for version 1.1+

You can merge this PR and I write another one to add the rights management process.

@cmb69
Copy link
Member

cmb69 commented Oct 15, 2024

Especially if the URL is going to change again, I suggest to add an entity declaration to https://github.com/php/doc-base/blob/master/entities/global.ent, so that translations don't have to update in the future.

@macintoshplus
Copy link
Contributor

Especially if the URL is going to change again, I suggest to add an entity declaration to https://github.com/php/doc-base/blob/master/entities/global.ent, so that translations don't have to update in the future.

Good idea.

I have open this PR to add the URL in entities: php/doc-base#164

I have integrated then entity usage in my PR #3871

@jimwins
Copy link
Member

jimwins commented Oct 16, 2024

@macintoshplus Can this be closed now that those other PRs have landed?

@cmb69
Copy link
Member

cmb69 commented Oct 16, 2024

Indeed, this PR is superseded by #3871.

@cmb69 cmb69 closed this Oct 16, 2024
@macintoshplus
Copy link
Contributor

@macintoshplus Can this be closed now that those other PRs have landed?

Yes. Thank you for your PR.

@mmalferov mmalferov deleted the patch-19 branch January 20, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA Quality Assurance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants