Skip to content

Commit

Permalink
Create congratulations bot (twentyhq#5404)
Browse files Browse the repository at this point in the history
- Created congratulations bot :
<img width="939" alt="Screenshot 2024-05-14 at 12 47 13"
src="https://github.com/twentyhq/twenty/assets/102751374/5138515f-fe4d-4c6d-9c7a-0240accbfca9">

- Modified OG image

- Added png extension to OG image route

To be noted: The bot will not work until the new API route is not
deployed. Please check OG image with Cloudflare cache.

---------

Co-authored-by: Ady Beraud <[email protected]>
  • Loading branch information
ady-beraud and ady-test authored May 21, 2024
1 parent 3deda2f commit 5ad59b5
Show file tree
Hide file tree
Showing 17 changed files with 277 additions and 73 deletions.
14 changes: 14 additions & 0 deletions .github/workflows/ci-utils.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ on:
# see: https://securitylab.github.com/research/github-actions-preventing-pwn-requests/
# and: https://github.com/facebook/react-native/pull/34370/files
pull_request_target:
types: [opened, synchronize, reopened, closed]
permissions:
actions: write
checks: write
Expand All @@ -19,6 +20,7 @@ concurrency:
jobs:
danger-js:
runs-on: ubuntu-latest
if: github.event.action != 'closed'
steps:
- uses: actions/checkout@v4
- name: Install dependencies
Expand All @@ -27,3 +29,15 @@ jobs:
run: cd packages/twenty-utils && npx nx danger:ci
env:
DANGER_GITHUB_API_TOKEN: ${{ github.token }}

congratulate:
runs-on: ubuntu-latest
if: github.event.action == 'closed' && github.event.pull_request.merged == true
steps:
- uses: actions/checkout@v4
- name: Install dependencies
uses: ./.github/workflows/actions/yarn-install
- name: Run congratulate-dangerfile.js
run: cd packages/twenty-utils && npx nx danger:congratulate
env:
DANGER_GITHUB_API_TOKEN: ${{ secrets.GITHUB_TOKEN }}
109 changes: 109 additions & 0 deletions packages/twenty-utils/congratulate-dangerfile.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,109 @@
import { danger } from 'danger';

const ordinalSuffix = (number) => {
const v = number % 100;
if (v === 11 || v === 12 || v === 13) {
return number + 'th';
}
const suffixes = { 1: 'st', 2: 'nd', 3: 'rd' };
return number + (suffixes[v % 10] || 'th');
};

const fetchContributorStats = async (username: string) => {
const apiUrl = `https://twenty.com/api/contributors/contributorStats/${username}`;

const response = await fetch(apiUrl);
const data = await response.json();
return data;
};

const fetchContributorImage = async (username: string) => {
const apiUrl = `https://twenty.com/api/contributors/${username}/og.png`;

await fetch(apiUrl);
};

const getTeamMembers = async () => {
const org = 'twentyhq';
const team_slug = 'core-team';
const response = await danger.github.api.teams.listMembersInOrg({
org,
team_slug,
});
return response.data.map((user) => user.login);
};

const runCongratulate = async () => {
const pullRequest = danger.github.pr;
const userName = pullRequest.user.login;

const staticExcludedUsers = [
'dependabot',
'cyborch',
'emilienchvt',
'Samox',
'charlesBochet',
'gitstart-app',
'thaisguigon',
'lucasbordeau',
'magrinj',
'Weiko',
'gitstart-twenty',
'bosiraphael',
'martmull',
'FelixMalfait',
'thomtrp',
'Bonapara',
'nimraahmed',
'ady-beraud',
];

const teamMembers = await getTeamMembers();

const excludedUsers = new Set([...staticExcludedUsers, ...teamMembers]);

if (excludedUsers.has(userName)) {
return;
}

const { data: pullRequests } =
await danger.github.api.rest.search.issuesAndPullRequests({
q: `is:pr author:${userName} is:closed repo:twentyhq/twenty`,
per_page: 2,
page: 1,
});

const isFirstPR = pullRequests.total_count === 1;

if (isFirstPR) {
return;
}

const stats = await fetchContributorStats(userName);
const contributorUrl = `https://twenty.com/contributors/${userName}`;

// Pre-fetch to trigger cloudflare cache
await fetchContributorImage(userName);

const message =
`Thanks @${userName} for your contribution!\n` +
`This marks your **${ordinalSuffix(
stats.mergedPRsCount,
)}** PR on the repo. ` +
`You're **top ${stats.rank}%** of all our contributors 🎉\n` +
`[See contributor page](${contributorUrl}) - ` +
`[Share on LinkedIn](https://www.linkedin.com/sharing/share-offsite/?url=${contributorUrl}) - ` +
`[Share on Twitter](https://www.twitter.com/share?url=${contributorUrl})\n\n` +
`![Contributions](https://twenty.com/api/contributors/${userName}/og.png)`;

await danger.github.api.rest.issues.createComment({
owner: danger.github.thisPR.owner,
repo: danger.github.thisPR.repo,
issue_number: danger.github.thisPR.pull_number,
body: message,
});
};

if (danger.github && danger.github.pr.merged) {
runCongratulate();
}
1 change: 1 addition & 0 deletions packages/twenty-utils/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
"scripts": {
"nx": "NX_DEFAULT_PROJECT=twenty-front node ../../node_modules/nx/bin/nx.js",
"danger:ci": "danger ci --use-github-checks --failOnErrors",
"danger:congratulate": "danger ci --dangerfile ./congratulate-dangerfile.ts --use-github-checks --failOnErrors",
"release": "node release.js"
}
}
4 changes: 2 additions & 2 deletions packages/twenty-website/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
"build": "npx next build",
"start": "npx next start",
"lint": "npx next lint",
"github:sync": "npx tsx src/github/github-sync.ts --pageLimit 1",
"github:init": "npx tsx src/github/github-sync.ts",
"github:sync": "npx tsx src/github/github-sync.ts",
"github:init": "npx tsx src/github/github-sync.ts --isFullSync",
"database:migrate": "npx tsx src/database/migrate-database.ts",
"database:generate:pg": "npx drizzle-kit generate:pg --config=src/database/drizzle-posgres.config.ts"
},
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ interface ProfileProps {

export const ProfileSharing = ({ username }: ProfileProps) => {
const [loading, setLoading] = useState(false);
const baseUrl = `${window.location.protocol}//${window.location.host}`;
const baseUrl = 'https://twenty.com';
const contributorUrl = `${baseUrl}/contributors/${username}`;

const handleDownload = async () => {
Expand Down Expand Up @@ -101,7 +101,7 @@ export const ProfileSharing = ({ username }: ProfileProps) => {
)}
</StyledButton>
<StyledButton
href={`http://www.twitter.com/share?url=${contributorUrl}`}
href={`https://www.twitter.com/share?url=${contributorUrl}`}
target="blank"
>
<XIcon color="black" size="24px" /> Share on X
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import { format } from 'date-fns';
import { ImageResponse } from 'next/og';

import {
bottomBackgroundImage,
backgroundImage,
container,
contributorInfo,
contributorInfoBox,
Expand All @@ -15,8 +15,7 @@ import {
profileInfoContainer,
profileUsernameHeader,
styledContributorAvatar,
topBackgroundImage,
} from '@/app/api/contributors/og-image/[slug]/style';
} from '@/app/api/contributors/[slug]/og.png/style';
import { getContributorActivity } from '@/app/contributors/utils/get-contributor-activity';

const GABARITO_FONT_CDN_URL =
Expand All @@ -33,8 +32,10 @@ const getGabarito = async () => {
export async function GET(request: Request) {
try {
const url = request.url;

const username = url.split('/')?.pop() || '';
const splitUrl = url.split('/');
const usernameIndex =
splitUrl.findIndex((part) => part === 'contributors') + 1;
const username = splitUrl[usernameIndex];

const contributorActivity = await getContributorActivity(username);
if (contributorActivity) {
Expand All @@ -45,11 +46,11 @@ export async function GET(request: Request) {
activeDays,
contributorAvatar,
} = contributorActivity;
return await new ImageResponse(

const imageResponse = await new ImageResponse(
(
<div style={container}>
<div style={topBackgroundImage}></div>
<div style={bottomBackgroundImage}></div>
<div style={backgroundImage}></div>
<div style={profileContainer}>
<img src={contributorAvatar} style={styledContributorAvatar} />
<div style={profileInfoContainer}>
Expand All @@ -59,8 +60,8 @@ export async function GET(request: Request) {
</h2>
</div>
<svg
width="96"
height="96"
width="134"
height="134"
viewBox="0 0 136 136"
fill="none"
xmlns="http://www.w3.org/2000/svg"
Expand Down Expand Up @@ -122,6 +123,7 @@ export async function GET(request: Request) {
],
},
);
return imageResponse;
}
} catch (error) {
return new Response(`error: ${error}`, {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,42 +14,36 @@ export const container: CSSProperties = {
fontFamily: 'Gabarito',
};

export const topBackgroundImage: CSSProperties = {
backgroundImage: `url(${BACKGROUND_IMAGE_URL})`,
export const backgroundImage: CSSProperties = {
position: 'absolute',
zIndex: '-1',
width: '1300px',
height: '250px',
transform: 'rotate(-11deg)',
opacity: '0.2',
top: '-100',
left: '-25',
};

export const bottomBackgroundImage: CSSProperties = {
backgroundImage: `url(${BACKGROUND_IMAGE_URL})`,
position: 'absolute',
zIndex: '-1',
width: '1300px',
height: '250px',
transform: 'rotate(-11deg)',
opacity: '0.2',
bottom: '-120',
right: '-40',
width: '1250px',
height: '850px',
transform: 'rotate(-7deg)',
opacity: '0.8',
backgroundImage: `
linear-gradient(
158.4deg,
rgba(255, 255, 255, 0.8) 30.69%,
#FFFFFF 35.12%,
rgba(255, 255, 255, 0.8) 60.27%,
rgba(255, 255, 255, 0.64) 38.88%
),
url(${BACKGROUND_IMAGE_URL})`,
};

export const profileContainer: CSSProperties = {
display: 'flex',
flexDirection: 'row',
justifyContent: 'space-between',
width: '780px',
margin: '0px 0px 40px',
width: '970px',
height: '134px',
margin: '0px 0px 55px',
};

export const styledContributorAvatar = {
display: 'flex',
width: '96px',
height: '96px',
width: '134px',
height: '134px',
margin: '0px',
border: '3px solid #141414',
borderRadius: '16px',
Expand All @@ -65,7 +59,7 @@ export const profileInfoContainer: CSSProperties = {

export const profileUsernameHeader: CSSProperties = {
margin: '0px',
fontSize: '28px',
fontSize: '39px',
fontWeight: '700',
color: '#141414',
fontFamily: 'Gabarito',
Expand All @@ -74,7 +68,7 @@ export const profileUsernameHeader: CSSProperties = {
export const profileContributionHeader: CSSProperties = {
margin: '0px',
color: '#818181',
fontSize: '20px',
fontSize: '27px',
fontWeight: '400',
};

Expand All @@ -84,8 +78,8 @@ export const contributorInfoContainer: CSSProperties = {
display: 'flex',
flexDirection: 'row',
justifyContent: 'space-around',
width: '780px',
height: '149px',
width: '970px',
height: '209px',
backgroundColor: '#F1F1F1',
};

Expand All @@ -110,21 +104,21 @@ export const contributorInfoTitle = {
color: '#B3B3B3',
margin: '0px',
fontWeight: '500',
fontSize: '24px',
fontSize: '33px',
};

export const contributorInfoStats = {
color: '#474747',
margin: '0px',
fontWeight: '700',
fontSize: '40px',
fontSize: '55px',
};

export const infoSeparator: CSSProperties = {
position: 'absolute',
right: 0,
display: 'flex',
width: '2px',
height: '85px',
height: '120px',
backgroundColor: '#141414',
};
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
import { getContributorActivity } from '@/app/contributors/utils/get-contributor-activity';
import { executePartialSync } from '@/github/execute-partial-sync';

export const dynamic = 'force-dynamic';

export async function GET(request: Request) {
try {
const url = request.url;

const username = url.split('/')?.pop() || '';

await executePartialSync();

const contributorActivity = await getContributorActivity(username);

if (contributorActivity) {
const mergedPRsCount = contributorActivity.mergedPRsCount;
const rank = contributorActivity.rank;
return Response.json({ mergedPRsCount, rank });
}
} catch (error: any) {
return new Response(`Contributor stats error: ${error?.message}`, {
status: 500,
});
}
}
Loading

0 comments on commit 5ad59b5

Please sign in to comment.