Skip to content

Explicitly tell user to start the service#277

Merged
imobachgs merged 1 commit intoagama-project:masterfrom
lkocman:master
Oct 28, 2022
Merged

Explicitly tell user to start the service#277
imobachgs merged 1 commit intoagama-project:masterfrom
lkocman:master

Conversation

@lkocman
Copy link
Collaborator

@lkocman lkocman commented Oct 27, 2022

Problem

This explicit mention of starting the service will save one step to the first-timer:

The readme says ./setup.sh will do all the work, and then simply point to http://localhost:9090/cockpit/@localhost/d-installer/index.html#/products. But then you login to cockpit and get "unable to connect to dbus".

Setup.sh tells user to run the service

ln -s `pwd`/dist ~/.local/share/cockpit/d-installer
/home/lkocman/Downloads/d-installer

Start the d-installer service:
  cd service; sudo bundle exec bin/d-installer

Visit http://localhost:9090/cockpit/@localhost/d-installer/index.html

But since it printed also "visit ... sentence" right after, I expected that perhaps that was also covered by setup.sh and it's simply just printed by some subcommand.

Solution

Add explicit task to start the service to README.md

Testing

n/a just a nitpicking on docs.

Screenshots

dbus

Copy link
Contributor

@imobachgs imobachgs left a comment

Choose a reason for hiding this comment

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

It makes sense. Thanks!

@imobachgs imobachgs merged commit b1d519d into agama-project:master Oct 28, 2022
@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.386% when pulling 783ce79 on lkocman:master into fa6eba5 on yast:master.

@imobachgs imobachgs mentioned this pull request Nov 16, 2022
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