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

Add uninstallation steps #1331

Merged
merged 7 commits into from
Jan 19, 2024
Merged

Add uninstallation steps #1331

merged 7 commits into from
Jan 19, 2024

Conversation

lidiazuin
Copy link
Contributor

No description provided.

. Use the preferred package manager and write:
[source]
---
sudo dpkg -r neo4j
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be sudo apt remove neo4j


In case you want to specifically remove Neo4j Desktop, refer to https://neo4j.com/docs/desktop-manual/current/troubleshooting/re-install/[Neo4j Desktop -> Re-installation of Desktop].

Follow these steps to uninstall Neo4j on Linux from a tarball:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think since this page is all about how to install using the deb package and the apt package manager, we can assume that any user here will be looking for uninstall instructions for deb. I think therefore we can just delete line 303

"Follow these steps to uninstall Neo4j on Linux from a tarball:"


[source]
---
yum remove <NEO4J HOME>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be sudo yum remove neo4j

Comment on lines 77 to 78
. Delete the _Neo4j Desktop.app_ from the Applications folder.
. Delete the _~/Library/Application Support/Neo4j Desktop_ and _~/.neo4j_ folders.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

earlier in the page we tell mac users to install with tar, so this will need the same instructions as that. I don't know if these extra steps about deleting something from the Applications folder are necessary, maybe check with someone who has a mac. @davidoliverSP2 perhaps?

modules/ROOT/pages/installation/windows.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/linux/tarball.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/windows.adoc Outdated Show resolved Hide resolved
Copy link
Contributor

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I added some editorial comments.


== Uninstall Neo4j

In case you want to specifically remove Neo4j Desktop, refer to https://neo4j.com/docs/desktop-manual/current/troubleshooting/re-install/[Neo4j Desktop -> Re-installation of Desktop].
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need to mention Neo4j Desktop on each OS page, as there is nothing related to how to install Desktop on any of them. So, I think you should remove this sentence.

modules/ROOT/pages/installation/linux/debian.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/linux/rpm.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/linux/rpm.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/linux/tarball.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/osx.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/osx.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/windows.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/windows.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/installation/linux/debian.adoc Outdated Show resolved Hide resolved
@neo-technology-commit-status-publisher
Copy link
Collaborator

Thanks for the documentation updates.

The preview documentation has now been torn down - reopening this PR will republish it.

Copy link
Contributor

@renetapopova renetapopova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks, @lidiazuin!

@lidiazuin lidiazuin merged commit aef2246 into neo4j:dev Jan 19, 2024
5 checks passed
@lidiazuin lidiazuin deleted the uninstall branch January 19, 2024 14:15
lidiazuin added a commit to lidiazuin/docs-operations that referenced this pull request Jan 19, 2024
lidiazuin added a commit to lidiazuin/docs-operations that referenced this pull request Jan 19, 2024
lidiazuin added a commit to lidiazuin/docs-operations that referenced this pull request Jan 19, 2024
lidiazuin added a commit that referenced this pull request Jan 19, 2024
Cherry-picked from #1331

Co-authored-by: Reneta Popova <[email protected]>
lidiazuin added a commit that referenced this pull request Jan 19, 2024
Cherry-picked from #1331

Co-authored-by: Reneta Popova <[email protected]>
lidiazuin added a commit that referenced this pull request Jan 19, 2024
Cherry-picked from #1331

Co-authored-by: Reneta Popova <[email protected]>
lidiazuin added a commit to lidiazuin/docs-operations that referenced this pull request Jan 19, 2024
lidiazuin added a commit to lidiazuin/docs-operations that referenced this pull request Jan 19, 2024
lidiazuin added a commit to lidiazuin/docs-operations that referenced this pull request Jan 19, 2024
lidiazuin added a commit that referenced this pull request Jan 19, 2024
Cherry-picked #1331

Co-authored-by: Reneta Popova <[email protected]>
lidiazuin added a commit that referenced this pull request Jan 19, 2024
Cherry-picked from #1331

Co-authored-by: Reneta Popova <[email protected]>
lidiazuin added a commit that referenced this pull request Jan 19, 2024
Cherry-picked from #1331

Co-authored-by: Reneta Popova <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants