Skip to content
New issue

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

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

Already on GitHub? # to your account

feat(api): event routes #1

Merged
merged 10 commits into from
Sep 24, 2023
Merged

feat(api): event routes #1

merged 10 commits into from
Sep 24, 2023

Conversation

Ethan-Chew
Copy link
Contributor

No description provided.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Sep 24, 2023

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 510ece6
Status: ✅  Deploy successful!
Preview URL: https://41204b41.sstaa.pages.dev
Branch Preview URL: https://eventapi.sstaa.pages.dev

View logs

server/api/event/[id].delete.ts Show resolved Hide resolved
server/api/event/[id].delete.ts Outdated Show resolved Hide resolved
server/api/event/[id].post.ts Outdated Show resolved Hide resolved
server/api/event/[id].post.ts Outdated Show resolved Hide resolved
server/api/event/[id].get.ts Outdated Show resolved Hide resolved
server/api/event/index.post.ts Outdated Show resolved Hide resolved
server/api/event/index.post.ts Outdated Show resolved Hide resolved
server/api/event/index.post.ts Outdated Show resolved Hide resolved
@qin-guan
Copy link
Member

Thanks for the work, overall lgtm.

I think to be consistent with rest of APIs, we can allow extraneous properties but just ignore them. Will make the necessary changes no worries

@qin-guan qin-guan changed the title EventAPI feat: event api Sep 24, 2023
@qin-guan qin-guan changed the title feat: event api feat(api): event routes Sep 24, 2023
@qin-guan qin-guan merged commit 1271638 into main Sep 24, 2023
1 check passed
@qin-guan qin-guan deleted the EventAPI branch September 24, 2023 15:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants