Skip to content
This repository has been archived by the owner on Jun 9, 2023. It is now read-only.

docs(dev): Add commands for banned and test users #1934

Closed
wants to merge 2 commits into from
Closed

docs(dev): Add commands for banned and test users #1934

wants to merge 2 commits into from

Conversation

jaitjacob
Copy link
Contributor

  • I have read Chapter's contributing guidelines.
  • My pull request has a descriptive title (not a vague title like Update README.md).
  • My pull request targets the main branch of Chapter.

I have referred to the new commands added in #1855 and updated the same in the docs.
Closes #1914

@jaitjacob jaitjacob requested a review from a team November 19, 2022 16:02
@gitpod-io
Copy link

gitpod-io bot commented Nov 19, 2022

@ghost
Copy link

ghost commented Nov 19, 2022

👇 Click on the image for a new way to code review

  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

gikf
gikf reviewed Nov 20, 2022
View reviewed changes
Copy link
Member

@gikf gikf left a comment

Choose a reason for hiding this comment

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

I should be more precise, there's few commands that will use the users seeded in the database

chapter/package.json

Lines 30 to 33 in 9ee629e

"change-user:banned": "npm run change-user banned@chapter.admin",
"change-user:test": "npm run change-user test@user.org",
"change-user:admin": "npm run change-user admin@of.chapter.one",
"change-user:owner": "npm run change-user foo@bar.com",

But to login to an account that isn't among those accounts, the npm run change-user <email> can be used.

chapter/package.json

Line 29 in 9ee629e

"change-user": "node scripts/change-user.js",

If there isn't yet account with that email, it will be created.

@Sboonny
Copy link
Member

Sboonny commented Nov 20, 2022

Hi @jaitjacob, thank you for interest in fixing this issue, but I have to close it because the issue is for people who are first timers to our codebase.

@Sboonny Sboonny closed this Nov 20, 2022
@Sboonny Sboonny mentioned this pull request Nov 20, 2022
Open
@jaitjacob
Copy link
Contributor Author

@Sboonny sure, np, I'll find something else

@jaitjacob jaitjacob deleted the docs/add-commands-for-banned-and-test-users branch November 20, 2022 13:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@gikf gikf gikf left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Updating Different Users section in contribution guide
3 participants
@jaitjacob @Sboonny @gikf