Skip to content

Fix "repository" link in README.md #935

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

Merged
merged 1 commit into from
May 27, 2024
Merged

Fix "repository" link in README.md #935

merged 1 commit into from
May 27, 2024

Conversation

derekmd
Copy link
Contributor

@derekmd derekmd commented May 26, 2024

Summary

The official documentation "repository" link is still leading to the old archived repo https://github.com/owen-it/laravel-auditing-doc/. Update to the currently-maintained https://github.com/owen-it/laravel-auditing.com/.

The official documentation "repository" link
is still leading to the old archived repo
https://github.com/owen-it/laravel-auditing-doc/.
Update to the currently-maintained
https://github.com/owen-it/laravel-auditing.com/.
@willpower232
Copy link
Contributor

Great spot, thank you!

@willpower232 willpower232 merged commit af71f96 into owen-it:master May 27, 2024
@erikn69
Copy link
Contributor

erikn69 commented May 27, 2024

@willpower232 Hi, use "Squash and Merge" in future merges

image

@willpower232
Copy link
Contributor

@erikn69 I was going to but github did not give me the option for some reason 🤔

@erikn69
Copy link
Contributor

erikn69 commented May 27, 2024

It's strange, I just display the list and select the option

image

@willpower232
Copy link
Contributor

I think github would have preferred me to click on the Approve And Run button to unleash the full experience or whatever, definitely a massive fan of squash merging. The other PRs have the dropdown so will double check in the future 🙏

@derekmd derekmd deleted the docs/readme-link-to-new-repo branch May 27, 2024 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants