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

Add option to change the type of user #1349

Merged
merged 14 commits into from
Mar 4, 2024
Merged

Conversation

nilsreichardt
Copy link
Member

@nilsreichardt nilsreichardt commented Mar 1, 2024

Description

A user can change twice every two weeks their type of user in the app setting without contacting the support.

Demo

Changing type of user

Screen.Recording.2024-03-03.at.17.46.57.mov

When the user changed too often the type of user

Screen.Recording.2024-03-03.at.17.48.48.mov

Related tickets

Closes #556

@github-actions github-actions bot added platform: macos feature: analytics Includes everything that is related to analytics and usage tracking labels Mar 1, 2024
Copy link

github-actions bot commented Mar 1, 2024

Visit the preview URL for this PR (updated for commit 72a584b):

https://sharezone-test--pr1349-change-account-type-61q55fat.web.app

(expires Thu, 07 Mar 2024 19:59:18 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: 4cb3ae61e1e018abfd9841fd3239f5b49ccc034b

@nilsreichardt nilsreichardt marked this pull request as ready for review March 3, 2024 16:54
import 'package:cloud_functions/cloud_functions.dart';
import 'package:user/user.dart';

class ChangeTypeOfUserRepository {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Its not really a Repository, lets just call it ChangeTypeOfUserService instead I'd say.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Or anything else, just not Repository since it does not follow the Repository pattern.

@Jonas-Sander
Copy link
Collaborator

Don't forget to change the PR if you implement my suggestion on the backend (errorId)

@nilsreichardt nilsreichardt enabled auto-merge March 4, 2024 19:21
@nilsreichardt nilsreichardt added this pull request to the merge queue Mar 4, 2024
Merged via the queue into main with commit 18b3d0e Mar 4, 2024
33 checks passed
@nilsreichardt nilsreichardt deleted the change-account-type branch March 4, 2024 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature: analytics Includes everything that is related to analytics and usage tracking platform: macos testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

User should be able to change the user type without contacting the support
2 participants