After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 722151 - Addressbook functionality should be documented in user help
Addressbook functionality should be documented in user help
Status: RESOLVED FIXED
Product: gnome-user-docs
Classification: Core
Component: gnome-help
3.11.x
Other Linux
: Normal normal
: ---
Assigned To: Maintainers of Gnome user documentation
Maintainers of Gnome user documentation
winterdocs
Depends on:
Blocks:
 
 
Reported: 2014-01-14 02:15 UTC by Michael Catanzaro
Modified: 2017-04-12 13:55 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Fix bug 722151 - update Contacts first run (2.16 KB, patch)
2017-03-19 19:18 UTC, Paul Cutler
none Details | Review
Fix bug 722151 (1.35 KB, patch)
2017-03-20 23:23 UTC, Paul Cutler
none Details | Review
Update fix with feedback from comments (1.18 KB, patch)
2017-04-12 00:05 UTC, Paul Cutler
none Details | Review

Description Michael Catanzaro 2014-01-14 02:15:09 UTC
This is a companion to Bug #722150.

There doesn't seem to be any information on how addressbooks or addressbook switching works.  It'd be good to add a section to explain this.
Comment 1 Michael Catanzaro 2014-04-08 15:52:55 UTC
In Bug #722150, Allan posted an explanation of how addressbook switching works: "New contacts will be added to the selected address book. You are able to view but not edit contacts from other address books."
Comment 2 Paul Cutler 2017-03-19 19:18:13 UTC
Created attachment 348270 [details] [review]
Fix bug 722151 - update Contacts first run

Add the following new sentence to Starting contacts section:
"All new contacts you create will be saved to the address book you choose.  You are also able to view and edit contacts in other address books." This fixes bug 722151 when a user might have an online account configured with contacts and a local address book.  I confirmed users can edit their online account contacts even after choosing a local addressbook at first run.
Comment 3 Petr Kovar 2017-03-20 21:58:12 UTC
Comment on attachment 348270 [details] [review]
Fix bug 722151 - update Contacts first run

Just a few nits.


>Add the following new sentence to Starting contacts section:
>"All new contacts you create will be saved to the address book you choose.  You are also able to view and edit contacts in other address books." This fixes bug 722151 when a user might have an online account configured with contacts and a local address book.  I confirmed users can edit their online account contacts even after choosing a local addressbook at first run.


It's better to wrap these lines at 72 characters. See https://wiki.gnome.org/Git/CommitMessages.


>@@ -29,7 +36,9 @@
>    
>   <p>If you have <link xref="accounts">online accounts</link> configured, they
>   are listed with <gui>Local Address Book</gui>. Select an item from the
>-  list and click <gui style="button">Done</gui>.</p>
>+  list and click <gui style="button">Done</gui>.  All new contacts you create
>+  will be saved to the address book you choose.  You are also able to view and
>+  edit contacts in other address books.</p>


Remove the double spaces at the end of sentences.
Comment 4 Paul Cutler 2017-03-20 23:23:56 UTC
Created attachment 348367 [details] [review]
Fix bug 722151

Add the following new sentence to Starting contacts section:
All new contacts you create will be saved to the address book you choose.
 You are also able to view and edit contacts in other address books."
Add feedback from comment 3 in the bug to only include single space when
starting a new sentence.
Comment 5 Petr Kovar 2017-04-10 16:12:39 UTC
Paul, could you squash the two patches into one with:

git rebase -i HEAD~2

Run in the branch where your patches are applied? Thanks!
Comment 6 Paul Cutler 2017-04-12 00:05:18 UTC
Created attachment 349700 [details] [review]
Update fix with feedback from comments

Thanks for the feedback.  I did git rebase -i HEAD~2, but git bz wouldn't work after that.  I then fixed a contraction in the last paragraph and did another commit, but git bz wouldn't apply the patch, so adding it manually.  Let me know if it doesn't apply and I can try again.