Update RTM to Socket in BotManFactory method name

Open
#70 0 comments 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
35/100
Issue type
Documentation
Clarity
Mostly clear
Activity status
Stale
Domain
documentation

Research direction

Start with driver-slack.md and review the BotManFactory::createForRTM reference against BotManFactory::createForSocket. Then inspect the driver-slack src directory and its file and class names; done means the documentation and any required names consistently use Socket terminology.

Written by the indexing model from the issue text.

Description

I think you might have forgotten to update the method name in https://github.com/botman/docs/blob/master/driver-slack.md from BotManFactory::createForRTM to BotManFactory::createForSocket. Maybe the file and class names in https://github.com/botman/driver-slack/tree/master/src also need updating?

Dominant language
No language data
Stars
33
Forks
49
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from botman/docs

All issues in botman/docs

Similar issues

More Documentation issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.