mazipan / tanyaaja.in

❓Kumpulkan pertanyaan secara anonim dari siapa saja dengan mudah
https://tanyaaja.in
MIT License
138 stars 38 forks source link

REFACTOR: move to useMutation for update channel notification API #105

Closed madjiebimaa closed 1 year ago

madjiebimaa commented 1 year ago

Closes #112

Description

Find a list of APIs that use the method PUT/PATCH/POST and have additional conditions (success, error, etc) or need some results (isLoading, etc). APIs that have those criteria will be added a layer before accessing those APIs directly with useMutation.

Current Tasks

APIs:

vercel[bot] commented 1 year ago

Someone is attempting to deploy a commit to a Personal Account owned by @mazipan on Vercel.

@mazipan first needs to authorize it.

mazipan commented 1 year ago

If it's too big, let's make it smaller by merging one by one.

Wdyt? @madjiebimaa

I will create separate issue for each API, so you can create separate PR based on that

madjiebimaa commented 1 year ago

If it's too big, let's make it smaller by merging one by one.

Wdyt? @madjiebimaa

I will create separate issue for each API, so you can create separate PR based on that

so rather than having several commits in one pull request, it would be good things if I create one commit for one pull request?

mazipan commented 1 year ago

Yes, I prefer to make it smaller. It will be easier to review also for me.

vercel[bot] commented 1 year ago

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

Name Status Preview Comments Updated (UTC)
tanyaaja ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 23, 2023 1:42pm
mazipan commented 1 year ago

Conflict ya @madjiebimaa

madjiebimaa commented 1 year ago

Conflict ya @madjiebimaa

Okay mas, I try to resolve this