#44 Add Zinebot Operations page
Merged by rlengland. Opened by glb.
Unknown source master

Download 44.patch

Hey Richard. They say turnabout is fair play. So I guess this is your chance for revenge for my finding all those typos on your earlier PRs. :-)

Also, if you could actually try to follow the installation instructions and point out anything that might be hard to follow or that might need further explanation, that would be great. And if you could beta-test the various chatroom commands to make sure they work as intended and as described, that too would be helpful.

Thanks!

P.S. I think the following commands should work to pull my 'zinebot' branch down to your local repo if you want to run the './build.sh && ./preview.sh' command to test it out locally.

$ cd <your-local-fedora-magazine-repo>
$ git fetch https://pagure.io/forks/glb/fedora-magazine.git/ zinebot
$ git checkout FETCH_HEAD

Commands

In the "TIP" last sentence dimond -> diamond

What are the compose characters for the "because" and "therefore"
symbols?? -> and <- are sort of intuitive but those aren't until you
see them.

Can you sneak these in someplace? It took me "forever" to find
BECAUSE and THEREFORE. Perhaps another "TIP" after they are called
out in the text.

: "∵" U22235 # BECAUSE
: "∴" U22234 # THEREFORE

and just for completeness:

: "←" U2190 # LEFTWARDS ARROW
: "→" U2192 # RIGHTWARDS ARROW

Good catch. Thanks!

As for using the math 'because' and 'therefore' operators. I chose those
because they are in a unicode range that doesn't seem to get enlarged when
they are typed alone on a line like many of the other characters do. I'm
not sure what governs that behavior. You can customize what "hotkeys" you
want the bot to respond to by tweaking the hotkeys.pm file under
~/.local/opt/zinebot/commands if you want. These are just some defaults
that I picked and they aren't strictly necessary for the operation of the
bot. I am open to other suggestions though.

rebased onto b285654a74e64d5f0f958ef7e42e887143241c32

Hey Richard. I just now tried to fix the problems you found as well as breaking all the lines up so that the diffs will be easier to read in the future.

I rewrote the history a bit though so that it looks like there is only one commit and the mistakes will not show up in the history. I think you can just re-fetch the branch and check it out to see the revised version. If you get an error about merge problems though, you might need to go back to (checkout) the master branch first. I'm not sure.

I'll take a look and let you know. I may wait until I've done the full test. "real soon now". :-)

rebased onto c085e67edf22c82ffdfe5d1488f4f73c1d70129b

rebased onto 5ac83262300855ddc6e887d0c485023ad4b93a6b

rebased onto 306edd41dbd85b6fb59a7344a4b5ce9b06c0d385

Somehow I accidentally removed the list of card properties from the earlier revisions. It is fixed now. Just be sure that the top line from git log shows a commit number beginning with 306edd4 to be sure that you are looking at the right version.

Thanks.

rebased onto 3e38f5ab1ac76cc555665c6afe4f96801551d4ff

Arrg!! I found another error. Just be sure that the commit you are looking at in your local copy matches the last "rebased" line shown in the PR comments.

rebased onto 6da331b4ab7d11240b42ed9895725f1ab98ae36d

rebased onto 3f03a6a8fb8f43ad7ea83249b66863ff77d55c5c

in § show schedule
"The results are then list in the chatroom." Should be "...listed in the chatroom."

Installation succeeded but execution did not. Pls see Matrix message from me.

I've replied to your Matrix messages. Thanks!

rebased onto 75213e26a04d3723ffb9b4ba1af7e4bd495bf76c

in § show schedule
"The results are then list in the chatroom." Should be "...listed in the chatroom."

This should be corrected now. Thanks!

Pull-Request has been merged by rlengland

Metadata