Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Creator Dashboard and Analytics #1492

Draft
wants to merge 21 commits into
base: main
Choose a base branch
from
Draft

Creator Dashboard and Analytics #1492

wants to merge 21 commits into from

Conversation

mdmohsin7
Copy link
Collaborator

@mdmohsin7 mdmohsin7 commented Dec 6, 2024

  • Capture Creator Details
  • Capture Payment Details
  • Remove name and email stuff from submit/update app
  • Creators dashboard screen (atleast a basic one)
  • Payments history
  • Overall apps analytics
  • Add manual payments
  • Update apps endpoints to use pydantic model
ScreenRecording_12-09-2024.21-35-46_1.1.mp4

@mdmohsin7 mdmohsin7 changed the title Creator profile Creator Dashboard and Analytics Dec 8, 2024
@beastoin
Copy link
Collaborator

beastoin commented Dec 9, 2024

@mdmohsin7 man, what is this and why we need it ?

@beastoin
Copy link
Collaborator

beastoin commented Dec 9, 2024

Screenshot 2024-12-09 at 12 03 32 and pls fix this 🥹

@mdmohsin7
Copy link
Collaborator Author

It doesn't work only for you @beastoin XD
Sending on iMessage, pls check

Screenshot 2024-12-09 at 10 55 47 AM

@beastoin
Copy link
Collaborator

beastoin commented Dec 9, 2024

oops sry, i'm a Firefoxman. it works on safari so nvm 👀 @mdmohsin7

@mdmohsin7
Copy link
Collaborator Author

@mdmohsin7 man, what is this and why we need it ?

This is for app creators to submit their paypal details, check payouts, and some basic analytics

@mdmohsin7 mdmohsin7 marked this pull request as ready for review December 9, 2024 16:16
@mdmohsin7
Copy link
Collaborator Author

Up for review @beastoin

@mdmohsin7 mdmohsin7 marked this pull request as draft December 11, 2024 03:59
@mdmohsin7
Copy link
Collaborator Author

Back to draft because have to include the recent payment changes, otherwise submitting an app will break

@beastoin
Copy link
Collaborator

my thoughts:

why do we need a separated creator profile man? isn't the user already a creator - or a developer - as default ? just simply put some extra fields to Developer Settings, should we?

the current analytic-related features are not stable enough(and, too small - i mean the metric) to put it into a hard-to-updates place likes app, why don't we just start with a simple web portal ?

@mdmohsin7 hope that help 🌚

@mdmohsin7
Copy link
Collaborator Author

why do we need a separated creator profile man? isn't the user already a creator - or a developer - as default ? just simply put some extra fields to Developer Settings, should we?

As a simple separation between a normal user and someone who builds stuff for Omi. This is mainly to have their details like the public name, email and payment details and nothing much. The developer mode or settings has a different meaning associated with it, so to not confuse people and keep things simple I added it as a separate option. I'm fine with putting them in developer settings

the current analytic-related features are not stable enough(and, too small - i mean the metric) to put it into a hard-to-updates place likes app, why don't we just start with a simple web portal ?

The initial version is mainly to just collect details and payout history (so that they know when we sent out the money and stuff), I added 1-2 minor analytics stuff to make it look better. Web portal would make sense when we have the create/update apps functionality on the web so that the creators use it completely instead of having to switch between platforms. Think of this inside of the app similar to the Google Play Console App on Play Store (minimal and for on the go stuff) and the web app should be similar to play console on web with all the stuff. Can get rid of the analytics but the screen would look empty as we will only have payout history and creator details

Lmk what do you think of it @beastoin

Sign up for free to 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