Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bootstrap ShapeIt Github Bot #2

Merged
merged 18 commits into from
Nov 8, 2023
Merged

Bootstrap ShapeIt Github Bot #2

merged 18 commits into from
Nov 8, 2023

Conversation

Amzani
Copy link
Contributor

@Amzani Amzani commented Nov 3, 2023

How to try this

Pitch onboarding

  1. Create a new Pitch
  2. Add the pitch in a ShapeIt project board (example: https://github.com/orgs/shapeitapp/projects/2)

Scope onboarding

  1. Add a scope as usual to a Pitch by using the related to #NUMBER
  2. An onboarding comment should appear on the scope

Note: If related to is absent in the scope no onboarding comment is shown for now.

Scope progress

  1. Comment /progress XX ... as usual
  2. The initial scope comment gets updated
  3. The original progress comment get minimised (not deleted for now)

Demo: shapeitapp/shapeitapp#28

Some details

Fixes shapeitapp/shapeitapp#34 shapeitapp/shapeitapp#28 shapeitapp/shapeitapp#25

@Amzani Amzani requested a review from fmvilas November 3, 2023 18:20
@Amzani Amzani changed the title Pitch message onboarding Bootstrap ShapeIt Github Bot Nov 4, 2023
Copy link
Contributor

@fmvilas fmvilas left a comment

Choose a reason for hiding this comment

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

Super pumped this is coming! 🙌

Left some comments but it's mostly about cleaning console.logs 😄

index.js Outdated Show resolved Hide resolved
index.js Outdated Show resolved Hide resolved
index.js Outdated Show resolved Hide resolved
index.js Outdated Show resolved Hide resolved
lib/onboarding.js Show resolved Hide resolved
lib/onboarding.js Outdated Show resolved Hide resolved
lib/progress.js Outdated Show resolved Hide resolved
lib/progress.js Outdated Show resolved Hide resolved
lib/scopes.js Outdated Show resolved Hide resolved
lib/scopes.js Outdated Show resolved Hide resolved
@Amzani Amzani requested a review from fmvilas November 6, 2023 16:31
@Amzani
Copy link
Contributor Author

Amzani commented Nov 6, 2023

@fmvilas thanks for your review, I've fixed a couple of stuffs

@Amzani
Copy link
Contributor Author

Amzani commented Nov 7, 2023

@fmvilas added deployment to fly.io / you should get an invitation by email.

@fmvilas fmvilas merged commit 171c77c into shapeitapp:main Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Setup GitHub App
2 participants