Skip to content

feat: rejectProject function, add userId to projects, add desktop and cli platforms, misc fixes - #20

Merged
loks0n merged 9 commits into
mainfrom
dev
Jun 5, 2023
Merged

feat: rejectProject function, add userId to projects, add desktop and cli platforms, misc fixes#20
loks0n merged 9 commits into
mainfrom
dev

Conversation

@loks0n

@loks0nloks0n commented Jun 2, 2023

Copy link
Copy Markdown
Member

What does this PR do?

  • adds 'desktop', 'cli', and 'other' platforms
  • fix bug in spacing between project tags
  • add userId to projects attributes
  • add some details to submitProject docs
  • add rejectProject function: sends an email to the project author when a rejectionReason is provided

Test Plan

Manuel

Have you read the Contributing Guidelines on issues?

Yes

@vercel

vercelBot commented Jun 2, 2023

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

NameStatusPreviewCommentsUpdated (UTC)
built-with-appwrite✅ Ready (Inspect)Visit Preview💬 Add feedbackJun 5, 2023 11:41am

Comment threadsrc/routes/projects/[projectId]/index.tsx
Comment threadsrc/Config.ts
Comment threadfunctions/rejectProject/src/index.js Outdated
Comment threadfunctions/rejectProject/src/index.js Outdated
Comment threadfunctions/rejectProject/src/index.js
Comment threadfunctions/rejectProject/src/index.js
Comment threadfunctions/rejectProject/src/index.js Outdated

@MeldironMeldiron left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Left review


List of environment variables used by this cloud function:

- **APPWRITE_FUNCTION_ENDPOINT** - Endpoint of Appwrite project

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Let'ß document APPROVED_EMAILS since we added it.

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Done

Sign up for freeto 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.

2 participants

@loks0n@Meldiron