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

Feature/onboarding #1296

Merged
merged 2 commits into from
Oct 16, 2022
Merged

Feature/onboarding #1296

merged 2 commits into from
Oct 16, 2022

Conversation

herpaderpaldent
Copy link
Contributor

@herpaderpaldent herpaderpaldent commented Oct 16, 2022

You must add a new environmental variable to your .env

ONBOARDING=true

When doing so new users (newer then 60 Minutes old) are required to complete the onboarding.
The steps are the following:

  1. Welcome
  2. (optional) open user-level enlistments
  3. Add additional characters
  4. (optional) open character-level enlistments

This way the experience for new users especially with open user-level enlistment is massively improved

image
image
image

herpaderpaldent and others added 2 commits October 16, 2022 20:38
You must add a new environmental variable to your `.env`
```
ONBOARDING=true
```

When doing so new users (newer then 60 Minutes old) are required to complete the onboarding.
The steps are the following:
1. Welcome
2. (optional) open user-level enlistments
3. Add additional characters
4. (optional) open character-level enlistments

This way the experience for new users especially with open user-level enlistment is massively improved
@codeclimate
Copy link

codeclimate bot commented Oct 16, 2022

Code Climate has analyzed commit 0e0a231 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 93.5% (50% is the threshold).

This pull request will bring the total coverage in the repository to 89.0% (0.0% change).

View more on Code Climate.

@herpaderpaldent herpaderpaldent merged commit 6876480 into 2.x Oct 16, 2022
@herpaderpaldent herpaderpaldent deleted the feature/onboarding branch October 16, 2022 18:49
@what-the-diff
Copy link

what-the-diff bot commented Oct 24, 2022

  • Added a new config option to enable the onboarding process

  • Created a migration for the onboardings table

  • Created an endpoint to resolve ids from names and vice versa

  • Implemented the first step of the onboarding process, which is adding all characters you want to track in Seat Plus (this will be used later on)

  • Implemented the second step of the onboarding process, which is enlistment into corporations/alliances/factions that are tracked by Seat Plus (this will be used later on)

  • Implemented a third step of the onboarding process, where users can add more characters they want to track with Seat Plus after finishing their initial setup (this will be used later on)

  • Added OnboardingMiddleware to redirect users that have not completed onboarding

  • Created a new controller for the onboarding process

  • Created a new view with all open enlistments and an introduction page

  • Removed the rejection of main character from user resource

  • Added onboarding model and migration

  • Added test for onboarding life cycle

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.

1 participant