Code Monkey home page Code Monkey logo

guestbook's Introduction

This repo is archived. If you are looking for the latest, check out open-sauced/insights or open-sauced/hot.


Open Sauced

πŸ• Open Sauced πŸ•

The path to your next Open Source contribution

CodeQL Compliance Release Publish stories if changed Netlify Status Dependabot Badge GitHub code size in bytes GitHub commit activity GitHub issues GitHub Release Discord Twitter

Open Sauced provides structured onboarding for new contributors to open source. This structure provides a way to track your next contributions by leveraging a unique dashboard built on top of the GitHub GraphQL API.

open-sauced-screencap

πŸ“– Prerequisites

In order to run the project from a container we need node>=14, npm>=7 and docker>=20 installed on our development machines or use one of the listed cloud providers we support:

Gitpod

βœ… fully ready integrated development environment

β­• VS Code settings sync is not ideal

Open in Gitpod

Gitpod is mostly free but has extensive documentation for all its contributing features.

For Vite to work with Gitpod we need to use a custom Dockerfile that lives under .gitpod.Dockerfile.

Open Sauced comes with a pre-defined .gitpod.yml that should work out of the box in multiple contribution scenarios.

Replit

βœ… fully ready integrated development environment

β­• modified contribution workflows

Open in Replit

Replit is an old service that recently supported NixOS, enabling full customisation of the runner and project.

Open Sauced is configured with a shell environment supporting node@16 and npm@8. The build environment script lives in shell.nix and the repl config lives in .replit.

Codesandbox

βœ… can be used as editor replacement

βœ… supports deployment preview

β­• enforces custom package manager for stability

β­• low resources free tier limitations

Open in CodeSandbox

Stackblitz

βœ… can be used as editor replacement

βœ… supports deployment preview

β­• enforces custom package manager for stability

β­• limited documentation on container limitations

Open in Stackblitz

Glitch

βœ… can be used as editor replacement

βœ… supports deployment preview

β­• enforces custom package manager for stability

β­• does not support stacked hot module reload

β­• severely low resources free tier limitations

Remix on Glitch

πŸ–₯️ Local development

To install the application:

npm ci

To start a local copy of the app on port 3000:

npm start

πŸ§ͺ Test

For running the test suite, use the following command. Since the tests run in watch mode by default, some users may encounter errors about too many files being open. In this case, it may be beneficial to install watchman.

npm test

You can request a coverage report by running the following command:

npm run test:coverage

πŸ“¦ Docker builds

A development preview can also be run from docker:

docker build -t open-sauced-dev .
docker run -p 8080:80 open-sauced-dev

Alternatively you can pull the production container and skip all builds:

docker run -dit -p 8080:80 ghcr.io/open-sauced/open-sauced

🎨 Code linting

To check the code and styles quality, use the following command:

npm run lint

This will also display during development, but not break on errors.

To fix the linting errors, use the following command:

npm run format

πŸš€ Production deployment

A production deployment is a complete build of the project, including the build of the static assets.

npm run build

You can analyze the build by running the following command:

npm run build:analyze

🎭 Offline asset optimization

We also have a script for updating .svg files used in the project, you only need to run this if you add new assets:

npm run build:svgo

And one for .png and .jpg files:

npm run build:squoosh

🚧 Development debugging

Thanks to antfu/vite-plugin-inspect we can always inspect the bundles in development by navigating to localhost:3000/__inspect/ in your browser.

πŸ“™ Storybook

Storybook is being leveraged to mock out visual React components. The latest version of the design system can be found at this URL.

npm run storybook

storybook example screenshot

πŸ”‘ Authentication

Authentication is handled through OneGraph's AuthGuardian service.

πŸ’Ύ Database

This project uses GitHub as a database. When you login, you will be presented with a button to create a goals repository. That repository template lives at open-sauced/goals-template.

πŸ’¨ Service Worker

This project uses the sw-precache to kickstart an offline cache. The offline cache only registers in production. If service needs to be manually removed make an unregister call from the registerServiceWorker.js import.

πŸŒ™ Dark Mode

This project supports "dark mode" styling, and by default it will follow the color preference on your device. It also allows for overriding this using buttons at the top right of the screen, which will persist the preference to local storage on your device. More info about color preference web API's can be found here: MDN Web Docs

πŸ“ Markdown Support

This project leverages Remirror for a delightful experience in documenting your Open Source goals. The editor supports markdown features including heading levels, bulleted lists, text formatting, code snippets, and emojis!

🀝 Contributing

We encourage you to contribute to Open Sauced! Please check out the Contributing guide for guidelines about how to proceed.

We have a commit utility called @open-sauced/conventional-commit that helps you write your commits in a way that is easy to understand and process by others.

It is generally integrated as an npm script but you can run it with npx as well:

npm run push

For any other npm based project or dotnpmrc defaulting to --yes:

npx -y @open-sauced/conventional-commit

πŸ• Community

Got Questions? Join the conversation in our Discord.
Find Open Sauced videos and release overviews on our YouTube Channel.

🎦 Repository Visualization

Visualization of this repository

βš–οΈ LICENSE

MIT Β© Open Sauced

guestbook's People

Contributors

adiati98 avatar ant-shell avatar at-the-vr avatar axfez avatar bekahhw avatar cbid2 avatar codewithjazzy avatar danieltott avatar izazw avatar jdwilkin4 avatar jmslynn avatar kamari93 avatar kelvinyelyen avatar macabonilas827 avatar muyixone avatar obasoro avatar pal-sandeep avatar piotrszymaniec avatar sadeedpv avatar salma-abdallah1 avatar samgkigotho avatar solenessa avatar sunny-unik avatar teemamin avatar tejasq avatar toobajamal avatar tpham20908 avatar vaibhavharsoda avatar virginielemaire avatar wasscodeur avatar

Stargazers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar  avatar

Watchers

 avatar  avatar  avatar  avatar

guestbook's Issues

Docs: Create an issue template to add users as contributors

Description

To get our "Intro to Open Source" course participants into the flow of contributing to open source, we want them to learn and experience creating an issue and link their PR to it later on.

Originally, we will let contributors to create an issue from scratch. However, adding an issue template would be a good practice for them to learn about issue's structure.

Based on these discussions:

Suggested Solution

To make it easier for contributors and maintainers to triage, let's create a simple issue template with below sections and add instructions (coment them out) to each section:

  • 1. Create an ISSUE_TEMPLATE folder inside .github folder.

  • 2. Create a file call feature_request.yml.

  • 3. Add these to the template (See our Feature Request template to help you out):

    • name: πŸš€ Feature request
    • description: Add contributors to the guest book πŸ’‘
    • title: "Feature: "
    • labels: [πŸ’‘ feature]
    • All have validations: required: true

    Atrributes' label in the body:

    • Description
      description: Please describe the issue β€” for example, "As part of the Intro to Open Source course, we are encouraged to contribute to this repo."

    • Suggested solution
      description: Suggest a solution β€” for example, "Add myself as a contributor."

    • Code of Conduct agreement

    • Getting Started Instructions agreement

      type: checkboxes
      id: contribution
      attributes:
        label: Getting Started Instructions
        description: Please read [Getting Started](https://intro.opensauced.pizza/#/05-how-to-contribute-to-open-source?id=getting-started) to add yourself as a contributor.
        options:
          - label: I agree to follow this project's Getting Started instructions.
            required: true

docs: Add extra instruction for manual entry and improve README

Description

When contributors add themselves manually, they should follow the emoji key and contribution types. Not following this list will break the process of generating README and the following PRs.

As there will be changes in the README, we might as well improve it.

Suggestion

  • Add an instruction as a warning to fill in the value strictly from the contribution types list.
  • Add OpenSauced's banner for consistency throughout repositories.
  • Change blob links to file's links.
  • Fix and adjust wordings for clarity if necessary.

Bug: CODEOWNERS does not automatically add reviewers

Describe the bug

Currently, we have CODEOWNERS file that contains the list of code owners. But they don't get added automatically as reviewers as it suppposed to. And it happens because the syntax has to be on the same line.

Steps to reproduce

Can't be reproduced without a new incoming PR.

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: Add @JulesRules65 as a contributor

Type of feature

πŸ“ Documentation

Current behavior

I want to do my first contribution to the roject

Suggested solution

I would like to add my name to the contribution list

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

fix: Make the greeting message more personalized

Type of feature

πŸ› Fix

Current behavior

I noticed that the message for first-time contributors is not personalized enough, which can make the experience a bit assembly-line-like.

Screenshot of first-time contributor greeting

Suggested solution

Instead of:
PR_COMMENT="Congratulations on completing Chapter 5 of the Intro to OSS Course with your contribution to this repository! You're almost to the end of the course. Create a highlight of your contribution to our guestbook using the instructions in chapter 6 and share it with us!"
curl -s -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" -X POST -d "{"body":"$PR_COMMENT"}" "https://api.github.com/repos/${{ github.repository }}/issues/${{ github.event.pull_request.number }}/comments”

How about:
PR_COMMENT="Congratulations on completing Chapter 5 of the Intro to OSS Course with your contribution to this repository @${{ github.actor }}! You're almost to the end of the course. Create a highlight of your contribution to our guestbook using the instructions in chapter 6 and share it with us!"
curl -s -H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" -X POST -d "{"body":"$PR_COMMENT"}" "https://api.github.com/repos/${{ github.repository }}/issues/${{ github.event.pull_request.number }}/comments"

It'll make the contributor feel more welcomed.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: adding contributor

Description

As part of the Intro to Open Source course, we are encouraged to contribute to this repo.

Suggested solution

Add myself as a contributor.

Code of Conduct

  • I agree to follow this project's Code of Conduct

Getting Started Instructions

  • I agree to follow this project's Getting Started instructions

Bug: remove extra `tr`

Describe the bug

We have an extra tr

image

Steps to reproduce

I haven't had a change to look at the code, but you should be able to find it in the readme. Remove the extra tr.

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

docs: more adjustments needed in `Contributing.md` file

Type of feature

πŸ“ Documentation

Current behavior

Based on our conversations in PR #202, there are more adjustments that the Contributing.md file needs in order to be guestbook-focused.

Suggested solution

The following section is needed for improvement:

Additional context

I want to work on this issue

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Docs: Update pull request template

Important

If you've worked on a good first issue in any of our community repositories before, please refrain from taking this issue.
We may remove you from the assignees and not accept your pull request if you choose to proceed.

Description

We have updated the "Getting Started" section on our Intro to Open Source course as following:

  • Adding contributors to the guestbook with CLI only.
  • Encouraging contributors to learn how to create issues.

Therefore, we need to update the PR template to reflect this update.

Additionally, we also want to remind contributors to complete the PR form.

Suggested Solution

  • Add a comment on the first line with the copy below.

    <!-- Please fill in all areas in this PR form. Incomplete PRs will be marked invalid and may be closed. -->
  • Add a new section for related issues right after "What type of PR" section. Use the copy below.

## Related Issues

<!-- 
Please use this format link issue numbers: Closes #123
More information: https://docs.github.com/en/free-pro-team@latest/github/managing-your-work-on-github/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword 
-->
  • Update wording in the "Contributing Checklist" section with the below suggestion.

    - ## Contributors checklist (check all applicable)
    + ## Contributors checklist

    ## Contributors checklist (check all applicable)

  • Remove the below sections.

    ### How did you add yourself as a contributor?
    - [ ] πŸ€– With CLI
    - [ ] ⌨️ Manually
    #### If you added yourself manually, did you follow the [emoji key and contribution types](https://allcontributors.org/docs/en/emoji-key) to fill in the value?
    - [ ] βœ… Yes
    - [ ] ❌ No

npm add command not working

Describe the bug

npm run contributors:add is not working as expected and is showing error in the terminal

Steps to reproduce

Fork and clone the repository and try npm run contributors:add command

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

docs: Clarity about screenshot

Type of feature

πŸ• Feature

Current behavior

Contributors have been skipping the screenshot section when filling out their PR form. It’s mostly likely due to assuming that’s it’s not required inspite of the course mentioning that it’s needed.

Suggested solution

  1. Go the .github folder
  2. Go to the Pull Request file
  3. On line 39, next to Screenshot, write "(*Required before Pull Request is Reviewed)"

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Docs: Remove the "Running the Project Locally" from Contributing Guide

Description

The content in the Running the Project Locally section in the Contributing Guide is meant for the intro repo.

This project is not built on Docsify.

Suggested solution

Remove the section from the Contributing Guideline.

Below are the code blocks for this section:

### Running the Project Locally
To contribute effectively to the OpenSauced project, it's essential to run the project locally to see your changes in action and test them thoroughly. We recommend two methods for running the project on your local machine: using the Docsify CLI or leveraging Visual Studio Code's Go Live feature.
#### Method 1: Using the Docsify CLI
1. **Install Node.js**: You can download it from the official Node.js [website](https://nodejs.org/.)
2. **Install Docsify Globally**: Open your command-line interface (CLI) and run the following command to install Docsify globally.
```bash
npm install -g docsify-cli
```
3. Run the project locally:
```bash
docsify serve
```
This command will start a local development server, and you can access the documentation in your web browser at `http://localhost:3000`
4. Make and Test Changes: With the local server running, you can make changes to the documentation files and immediately see the results in your browser. Test your changes thoroughly.
5. Stop the Server: When you're done testing or making changes, you can stop the local server by pressing `Ctrl + C` in the CLI.
#### Method 2: Using Visual Studio Code (VS Code) Go Live Feature
Once you have cloned the repository to your local machine, you can use VS Code's Go Live feature to run the project locally. This feature allows you to launch a local development server and preview your changes in real time in your browser.
1. **Install Visual Studio Code**: Download and install [Visual Studio Code](https://code.visualstudio.com/).
2. **Open the Project in VS Code**: Launch VS Code and open the project folder by selecting "File" > "Open Folder" and choosing the project directory.
3. **Install the Live Server Extension**: In VS Code, navigate to the Extensions view (you can press `Ctrl + Shift + X`) and search for "Live Server. Install the "Live Server" extension by Ritwick Dey.
4. **Launch the Live Server**: In the opened HTML file, you should see a "Go Live" button in the bottom-right corner of the VS Code window. Click this button to launch the live server.
This will open your documentation in a new browser tab at a local address (typically `http://127.0.0.1:5500`).
6. **Make and Test Changes**: With the Live Server running, you can make changes to the documentation files using VS Code, and the changes will be automatically reflected in your browser. Test your changes thoroughly.
7. **Stop the Live Server**: When you're done testing or making changes, you can stop the Live Server by clicking the "Exit" button in the VS Code status bar.

Bug: Unknown contribution type error when trying to run the npm run contributors:add command

Describe the bug

I tried running the npm run contributors:add command locally, and ran into this error

Error: Unknown contribution type web development for contributor

Here is the complete screenshot

Screenshot 2023-10-25 at 12 06 59 PM

The error is happening because web development is not a valid entry for the contributions array

 "contributions": [
        "web development"
      ]

the fix would be to change that to code since that is a valid entry

 "contributions": [
        "code"
      ]

But also, for the long term it might be nice to have an additional check in place to double check that the contributions array only holds valid values.
That way if a user tries to contribute to the guestbook manually with an invalid value, then the check will catch that.

Steps to reproduce

  1. got the guestbook repo locally
  2. pull down the latest changes from main
  3. create a new branch
  4. run the npm run contributors:add command
  5. see the error

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Docs: Remove "Adding a New Section to the Documentation" section

Description

We have a section in the CONTRIBUTING.md called Adding a New Section to the Documentation.

This section is no longer needed as it's specific for intro repository and we have removed the subsections from this repo's Contributing Guide.

Suggested solution

Remove the section from Contributing Guide.

Below is the code block for the section that needs to be removed:

## Adding a New Section to the Documentation
Welcome to the documentation contribution guide. In this guide, you will learn how to add a new section to our project's documentation. We have specific conventions for numbering sections to maintain a consistent structure.

Bug: message repeats when contributor edits PR

Describe the bug

Our bot is overly aggressive.

image

Steps to reproduce

See this PR #180

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: add @skippharaoh as a contributor

Description

As a part of the Intro to Open Source course, we are encouraged to practically practice open source by contributing to this repo.

Suggested solution

Add my username to this repo as a contributor.

Code of Conduct

  • I agree to follow this project's Code of Conduct

Getting Started Instructions

  • I agree to follow this project's Getting Started instructions

Feature: Issue for commits pushed to main branch

Type of feature

πŸ• Feature

Current behavior

There has been a recurring issue with contributors pushing their contributions to the repo’s main branch, making the project vulnerable to breaks.

Suggested solution

It would be best to add put a protection rule for the main branch.

Per GitHub’s documentation for branch protection rules, do the following:

  • Go to the "Code and automation" section of the sidebar in the , click Branches.
  • Go to Branches Protection Rules, click Add Branch Protection rule, and type main in Branch Name Pattern
  • In Protect matching branches, click on the Require a pull request before merging checkbox
  • Next, click on Required Approvals and in its dropdown menu, pick 3.
  • Click on Create

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

docs: Add screenshot requirement to PR template

Description

We received a handfull of PRs that missed the step of generating profile and badge on the README.

Proposed Solution

Add a section of "Screenshot" after the "Added to documentation?" to the PR template. There will be an explanation here that contributors need to provide a screenshot of their profile being generated on README. This will ease maintainers in reviewing PRs and ensure contributors to run the generate command.

feat: add translation rules and guide to README

          > @CBID2 I did look into Crowdin. I think we need to make sure there's interest. I'm super excited to have folks interested in translating, but we need to gauge how many people are _using_ the translated versions as part of the discussion. Maybe we should consider adding something to the PR template to ask them to select the language they used to complete the task.

That's a great idea! :) I do a PR for that part if you'd like.

Originally posted by @CBID2 in #79 (comment)

Feature: Add explanation to `git commit -am`

Type of feature

πŸ“ Documentation

Current behavior

Step 7 in the Getting Started section of the README file uses commit -am "Your message". However, it doesn't explain what that commit does. Since this repo is aimed at people who finished the intro course, contributors might not be too comfortable with using this command yet.

Suggested solution

Provide a brief explanation of this commit.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Bug: Incorrect links and term's consistency on the README

Description

There are incorrect links in the README. Also, we want to have consistency with the term "guestbook".

Suggested Solution

  • Change all "guest book" term to "guestbook" on the README. If it starts with capital letter, keep the capital letter.

  • Update links and some wording in the code snippet with the suggestion following the snippet:

    • This guest book is a place for people who have taken [OpenSauced](https://opensauced.pizza)'s [Intro to Open Source course](https://github.com/open-sauced/intro) to take their first steps into contributing to open source.

      This guestbook is a place for people who have taken [OpenSauced](https://app.opensauced.pizza/)'s [Intro to Open Source course](https://intro.opensauced.pizza/#/) to take their first steps into contributing to open source.
    • If you encounter merge conflicts while contributing to this repository, read our Intro to Open Source course's "[Merge Conflicts in the Guest Book Repository](https://intro.opensauced.pizza/#/05-how-to-contribute-to-open-source?id=merge-conflicts-in-the-guest-book-repository)" section.

      If you encounter merge conflicts while contributing to this repository, read our Intro to Open Source course's "[Merge Conflicts in the Guestbook Repository](https://intro.opensauced.pizza/#/05-how-to-contribute-to-open-source?id=merge-conflicts-in-the-guestbook-repository)" section.
    • You can add this contribution to your [OpenSauced](https://opensauced.pizza) profile as [outlined in the course](https://github.com/open-sauced/intro/blob/main/05-how-to-contribute-to-open-source.md).

      You can add this contribution to your [Highlight](https://app.opensauced.pizza/feed) at OpenSauced! Read our docs to learn more about [Highlight feature](https://docs.opensauced.pizza/features/highlights/).
    • Hungry for more contributions? Check out our [pizza verse](https://github.com/open-sauced/pizza) repository and join us in celebrating pizza!πŸ•πŸ˜„

      Hungry for more contributions? Check out our [pizza-verse](https://github.com/open-sauced/pizza-verse) repository and join us in celebrating pizza!πŸ•
    • All contributors are required to abide by our [Code of Conduct](https://github.com/open-sauced/.github/blob/main/CODE_OF_CONDUCT.md). Please follow the [README](README.md) directions for contributing.

      All contributors are required to abide by our [Code of Conduct](https://github.com/open-sauced/.github/blob/main/CODE_OF_CONDUCT.md). Please follow our [Contributing Guidelines](CONTRIBUTING.md) for contributing.
    • - **Course improvement**: If you are interested in improving OpenSauced's Intro to Open Source course, please create an issue at our [intro repository](https://github.com/open-sauced/intro) and refer to the [Contributing Guide](CONTRIBUTING.md).

      - **Course improvement**: If you are interested in improving OpenSauced's Intro to Open Source course, please create an issue at our [intro repository](https://github.com/open-sauced/intro) and refer to the [Contributing Guidelines](https://github.com/open-sauced/intro/blob/main/CONTRIBUTING.md).

Feature: Steps to edit the `.all-contributorsrc` file

Type of feature

πŸ• Feature

Current behavior

This repository is using CLI to add contributors to the README.md file.

However, there is also a suggestion in the README:

... we recommend actually editing the .all-contributorsrc file to get more experience with pull requests and reviews.

Does this mean that we are suggesting contributors to add themselves manually in the file?

Suggested solution

If we want to encourage contributors to add themselves manually to the file, we also want to provide instructions on how to do so.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

docs: more adjustments to the PR template

Type of feature

πŸ“ Documentation

Current behavior

When filling out the form for docs-related PRs, I found myself struggling with answering some of the questions and connecting the corresponding issue to it. A similar phenomenon could happen for people who want to do translation contributions or improve the directions in the README and Contributing files.

Suggested solution

  1. Put a [] Not applicable checkbox in the Contributors Checklist section
  2. Put a [] πŸ“ CONTRIBUTING.md checkbox in the Added to Documentation section.
  3. Put a ## Closes Issue section after the What Type of PR is this? section

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Bug: Step Typo in Step 7 of the Getting Started Section

Describe the bug

Currently, in the step 7 of the Getting Started section we have this:

7. If you used the CLI tool method in step 6, to add yourself to the guestbook, ...

It should be step "5" instead of "6".

This can be misleading, so we need to fix it.

Steps to reproduce

  1. Go to the Getting Started section.
  2. See step no. 7

Browsers

Other (add additional context)

Additional context (Is this in dev or production?)

N/A

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: Add a PR Template

Type of feature

πŸ• Feature

Current behavior

Currently, we're using the default PR template that lives in our .github repository for this repo.
However, there are some sections and points that aren't needed here, such as:

  • Related Tickets & Documents
  • Mobile & Desktop Screenshots/Recordings
  • Added tests?
  • [optional] Are there any post-deployment tasks we need to perform?

Suggested solution

Based on the default template, create a PR template in the .github folder with related contents for this repo:

  • Remove the sections that are mentioned above.
  • Include only necessary items in the "What type of PR is this?" section as follow:
    • Add a contributor
    • Documentation Update
  • Include only necessary items in the "Added to documentation?" section as follow:
    • README.md
    • no documentation needed
  • Add a new section called "Contributors checklist" to remind contributors of the steps that they should take:
    • Read the Getting Started section thoroughly.
    • How did they add themselves to the guestbook?
      • With CLI
      • Manually
    • If they added themselves manually, did they follow the emoji key and contribution types to fill in the values?
    • Have they run npm run contributors:generate?

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: add @chrisVCH as a contributor

Type of feature

πŸ• Feature

Current behavior

I want to make my first contribution.

Suggested solution

I would like to add myself as a contributor to the guestbook.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Bug: Translation Guide updates

Describe the bug

I noticed two issues in regards to the translation guide:

  1. The guide is not hyperlinked in second bullet point in the Contributing to this Repository section of the README file
  2. Thei18tn-guidelines.md file is labeled incorrectly.

Steps to reproduce

Issue 1

  1. Go to the second bullet point in the Contributing to this Repository section of the README file
  2. Try to click on community guidelines
  3. Nothing happens

Issue 2

  1. Go to the Code tab
  2. Go to thei18tn-guidelines.md file and see the wrong file name

Browsers

Other (add additional context)

Additional context (Is this in dev or production?)

N/A

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Bug: issue template for guestbook is not listed

Describe the bug

We have an issue template special to add contributors to guestbook repo. But this template is not listed when we click "New issue" button.

The problem is the ISSUE_TEMPLATE folder is located inside the .github/workflows and not on the root of .github folder.

We also want to rename the issue template to be add_contributor.yml to distinguish it from feature_request.yml template.

Suggested Solution

  • Move the ISSUE_TEMPLATE to the root of .github folder.
  • Rename issue template file to add_contributor.yml.
  • Update name in the template to reflect the change.

Steps to reproduce

  1. Click the "Issues" tab.
  2. Click "New issue" button.
  3. You won't find the issue template to add contributors.

Do you have any images or screen captures?

Screenshot 2024-03-26 110225

Browsers

No response

Additional context (Is this in dev or production?)

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: Add a Resolve Merge Conflicts section on README

Type of feature

πŸ• Feature

Current behavior

The process of adding contributors to this repository gives high potential of resolving conflicts before PR can be merged. It happens because new contributors will always add themselves at the end of the contributors array.

Suggested solution

  • Add a section in the README to walk through the process of resolving merge conflicts.
  • Add screeshots/screen recordings (GIF)/video walkthrough for better understanding.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: Translate guestbook directions into other languages

Type of feature

πŸ• Feature

Current behavior

The guestbook directions are written only in English, which can make new open source contributors, especially those who have English as a second language or do not speak it all.

Suggested solution

In the spirit of @antonio-pedro99's suggestion in issue #16 of the intro course's repo, I think it would be helpful to have the directions for the guestbook in Portuguese.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature: Add a section on how to solve Merge Conflicts to Contributing Guide

Type of feature

πŸ• Feature

Current behavior

A common issue that has been going with this repository is merge conflicts.

Suggested solution

It would be beneficial to add a section on updating branches and solving merge conflicts to the Contributing Guide. That way, contributors can access this information quickly and gain some practice.

As far as structure goes, I'm thinking of modeling after the subsection in the Pizza Verse Repo.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

docs: Move the content of "Getting Started" and "Update Branch & Resolve Conflicts" sections to Intro to Open Source course

Description

Outside of Hacktoberfest, most contributors who contribute to this repo are participants of our Intro to Open Source course. So I propose to remove the "Getting Started" and "Update Branch & Resolve Conflicts" sections and move them to the Intro to Open Source course.

Moving these sections to the course will give 2 benefits:

  1. As a one-stop-place of all course's docs for our participants.
  2. When folks who don't take the course come to this repo, they would be sent to our course. That way, it can attracts folks to check out our course and potentially, recommend the course to others.

Propose Solution

Note

This issue is depending on this PR in the intro repository.

feature: Add docs contributing guidelines

There are two types of contrubutions to this repo:

  1. adding yourself (according to chapter directions).
  2. updating the repo to make it a better contributor experience.

We've been successful with the docs for the first. Let's improve the docs for the second.

To do this, we need to explain how to run the project locally: https://github.com/open-sauced/intro/blob/main/CONTRIBUTING.md.

I'm in favor of adding this CONTRIBUTING.md file to our repo and linking in our readme with an explanation that if you're interested in contributing to improve this course, please create an issue and refer to this guide.

Feature: Clarification needed in Translation Guide

Type of feature

πŸ• Feature

Current behavior

The guestbook’s instructions has not been translated yet, so we needed to make some changes to the first section of the translation guide.

Suggested solution

  1. Go to the first section of the translation guide.
  2. In the part where it says, "At the moment, we have the course in the following languages:", place (To Be Announced)next to each bullet point.

Additional context

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct

Contributing Docs

  • I agree to follow this project's Contribution Docs

Feature:add @Woytsekj as a contributor

Description

As part of the Intro to Open Source course, we are encouraged to contribute to this repo.

Suggested solution

Add myself as a contributor

Code of Conduct

  • I agree to follow this project's Code of Conduct

Getting Started Instructions

  • I agree to follow this project's Getting Started instructions

Docs: Adjust wording in the Contributing Guide

Description

As we are adjusting our Contributing Guide to be only be about contributing to improve this repository, we would also adjust some of the wording in the guide.

Originally posted by @BekahHW in #202 (comment)

Suggested solution

  • Add this sentence as an opening line for the guide.

    This Contributing Guide will walk you through how to contribute to this project for bug fixes and features. If you want to contribute to the guestbook, please refer to chapter five of the Intro to Open Source course.

  • Fix the wording of step 3 in the "Setup the Project Locally" as below:

    Navigate to and open the project in your code editor.

    3. cd into the repository and open the project in your code editor.

  • Remove step 4 in the "Setup the Project Locally".

    4. Run the project locally. See the [Let's Get Practical](https://intro.opensauced.pizza/#/05-how-to-contribute-to-open-source?id=let39s-get-practical) section on the Intro to Open Source course for more information.

  • Update the numbering to reflect the changes

Method 2: Using Visual Studio Code (VS Code) Go Live Feature

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    πŸ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. πŸ“ŠπŸ“ˆπŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❀️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.