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

Fix typos and make minor edits #563

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Conversation

peterhauke
Copy link
Contributor

Fix typos and make minor edits.

The phrase "This B solution" seems to be in the wrong place or is no longer relevant, so I removed it. However, I am not sure about this.

I rewrote the sentences about why somebody might not like Backpack. I think it is better not to use that kind of negative phrasing in your documentation. The phrasing I have used is more neutral and concentrates of "use" rather than "not like".

Fix typos and make minor edits.

The phrase "This B solution" seems to be in the wrong place or is no longer relevant, so I removed it. However, I am not sure about this.

I rewrote the sentences about why somebody might not like Backpack. I think it is better not to use that kind of negative phrasing in your documentation. The phrasing I have used is more neutral and concentrates of "use" rather than "not like".
@pxpm
Copy link
Contributor

pxpm commented Apr 9, 2024

Actually that public publish step is completely removed in Backpack v6.

Backpack starting v6 uses https://github.com/Laravel-Backpack/basset

Can you remove the changes done in the publishing step and keep the others so that I can merge them, and I will find time to re-write that bit of docs.

The asset are in the files that use them as they include the cdn link.

Cheers

@peterhauke
Copy link
Contributor Author

@pxpm I hope that I have removed the changes you mentioned correctly. Thanks.

@pxpm pxpm self-assigned this Apr 10, 2024
@pxpm pxpm added the documentation Improvements or additions to documentation label Apr 10, 2024
@pxpm
Copy link
Contributor

pxpm commented Apr 10, 2024

Perfect @peterhauke

Really appreciate the effort 🙏

Cheers

@pxpm pxpm merged commit 813b12f into Laravel-Backpack:master Apr 10, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants