Skip to content
This repository has been archived by the owner on May 25, 2023. It is now read-only.

fix(onboarding-broke): new user onboarding was broken, sort of #1412

Merged
merged 1 commit into from
Apr 2, 2022
Merged

Conversation

bdougie
Copy link
Member

@bdougie bdougie commented Apr 1, 2022

What type of PR is this? (check all applicable)

  • πŸ• Feature
  • πŸ› Bug Fix
  • πŸ“ Documentation Update
  • 🎨 Style
  • πŸ§‘β€πŸ’» Code Refactor
  • πŸ”₯ Performance Improvements
  • βœ… Test
  • πŸ€– Build
  • πŸ” CI
  • πŸ“¦ Chore (Release)
  • ⏩ Revert

Description

This PR fixes the issue with new users unable to create goals repos. Turns out when a repo does not exist, the ternary was only checking for an error and this is technically not an error.

Related Tickets & Documents

fixes #1411

Mobile & Desktop Screenshots/Recordings

Added tests?

  • πŸ‘ yes
  • πŸ™… no, because they aren't needed
  • πŸ™‹ no, because I need help

Added to documentation?

  • πŸ“œ README.md
  • πŸ““ docs.opensauced.pizza
  • πŸ• dev.to/opensauced
  • πŸ“• storybook
  • πŸ™… no documentation needed

[optional] Are there any post-deployment tasks we need to perform?

[optional] What gif best describes this PR or how it makes you feel?

@bdougie bdougie requested a review from 0-vortex April 1, 2022 23:40
@0-vortex
Copy link
Contributor

0-vortex commented Apr 2, 2022

Seems to work fine on new user:
Screenshot 2022-04-02 at 16 22 27
Screenshot 2022-04-02 at 16 22 36
Screenshot 2022-04-02 at 16 22 44
Screenshot 2022-04-02 at 16 22 53
Screenshot 2022-04-02 at 16 23 14
Screenshot 2022-04-02 at 16 24 06

@bdougie bdougie merged commit 52eb2b5 into main Apr 2, 2022
@bdougie bdougie deleted the 1411 branch April 2, 2022 14:49
github-actions bot pushed a commit that referenced this pull request Apr 2, 2022
### [0.44.1](v0.44.0...v0.44.1) (2022-04-02)

### πŸ› Bug Fixes

* **onboarding-broke:** new user onboarding was broken, sort of ([#1412](#1412)) ([52eb2b5](52eb2b5))
@github-actions
Copy link
Contributor

github-actions bot commented Apr 2, 2022

πŸŽ‰ This PR is included in version 0.44.1 πŸŽ‰

The release is available on:

Your semantic-release bot πŸ“¦πŸš€

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: Onboarding is broken for users without a goals repo
2 participants