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

Several changes from me (Revise Fedora instructions, Add something on Debugging, and Remove the Zorin OS sections). #77

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Runa-Chin
Copy link
Contributor

Hello again, it's me Runa. I create the new pull request again for improve the Docs.

In this changes, I had create the revision and changes on Fedora sections which changes the option After installed...... from open the app menu into init it via commands. Besides that, I also revise little bit for Fedora sections and remove the Zorin OS sections due to archived repos for instructions and may be outdated. And finally, I add something on Debugging section for firewalld which added --permanent options on firewalld commands.

Please review it dear @aleasto and @electrikjesus, and feel free to commit it when something's nothing broken.

@aleasto
Copy link
Member

aleasto commented Sep 2, 2024

I had create the revision and changes on Fedora sections which changes the option After installed...... from open the app menu into init it via commands

Why?

@Runa-Chin
Copy link
Contributor Author

I had create the revision and changes on Fedora sections which changes the option After installed...... from open the app menu into init it via commands

Why?

For Fedora , I guess it should to change it from open app into enter the command to make it easier to launch the first time after installation using the CLI

@Runa-Chin
Copy link
Contributor Author

And severe people sometimes still confused what to do after installation. If you not agreed, I can revert it.

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.

2 participants