Skip to content

Add dashboard link to the codeball approve message#35

Merged
krlvi merged 2 commits intomainfrom
30-add-links-to-dashboard-from-approval-message
Jun 13, 2022
Merged

Add dashboard link to the codeball approve message#35
krlvi merged 2 commits intomainfrom
30-add-links-to-dashboard-from-approval-message

Conversation

@krlvi
Copy link
Member

@krlvi krlvi commented Jun 13, 2022

No description provided.

@krlvi
Copy link
Member Author

krlvi commented Jun 13, 2022

beta test: krlvi/bostats2#8

@krlvi krlvi requested review from ngalaiko and zegl June 13, 2022 11:55

const octokit = new Octokit({auth: githubToken})

const dashboardLink = '[stats](https://codeball.ai/' + process.env.GITHUB_REPOSITORY + ')'
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe call the link "dashboard"? To make it consistent with the website.

@zegl
Copy link
Contributor

zegl commented Jun 13, 2022

This action seems to be broken, the "message" input is never used. Could you take a look?

@krlvi krlvi force-pushed the 30-add-links-to-dashboard-from-approval-message branch from c4498ae to ac3afbf Compare June 13, 2022 13:41
@krlvi krlvi changed the title Add dashboard link to the default codeball approve message Add dashboard link to the codeball approve message Jun 13, 2022
@krlvi krlvi requested a review from zegl June 13, 2022 13:49
@krlvi krlvi merged commit 16642cd into main Jun 13, 2022
@krlvi krlvi deleted the 30-add-links-to-dashboard-from-approval-message branch June 13, 2022 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants