8000 fix(ux): re routes based on setup_complete by maharshivpatel · Pull Request #21118 · frappe/frappe · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix(ux): re routes based on setup_complete #21118

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

Merged
merged 5 commits into from
May 30, 2023

Conversation

maharshivpatel
Copy link
Contributor

If Setup Wizard is not complete re_route any route to setup_wizard and if it complete re_route setup-wizard to app

When setup is not complete.

Before.Setup.Wizard.is.complete.mov

When setup is complete.

After.Setup.Wizard.is.complete.mov

If Setup Wizard is not complete re_route any route to setup_wizard and if it complete re_route setup-wizard to app
@maharshivpatel maharshivpatel changed the title fix (UX): re routes based on setup_complete fix(ux): re routes based on setup_complete May 26, 2023
@codecov
Copy link
codecov bot commented May 26, 2023

Codecov Report

Merging #21118 (045415a) into develop (f6da683) will decrease coverage by 0.26%.
The diff coverage is 20.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #21118      +/-   ##
===========================================
- Coverage    63.98%   63.73%   -0.26%     
===========================================
  Files          765      765              
  Lines        69186    69622     +436     
  Branches      6251     6255       +4     
===========================================
+ Hits         44271    44371     +100     
- Misses       21374    21660     +286     
- Partials      3541     3591      +50     
Flag Coverage Δ
server-ui 32.13% <ø> (+<0.01%) ⬆️
ui-tests 51.51% <20.00%> (-0.37%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@maharshivpatel maharshivpatel marked this pull request as ready for review May 26, 2023 06:25
@maharshivpatel maharshivpatel requested review from a team and phot0n and removed request for a team May 26, 2023 06:25
@ankush ankush merged commit e6e59cb into frappe:develop May 30, 2023
@ankush ankush added the backport version-14-hotfix backport to version 14 label May 31, 2023
mergify bot pushed a commit that referenced this pull request May 31, 2023
* fix(ux): re routes based on setup_complete

If Setup Wizard is not complete re_route any route to setup_wizard and if it complete re_route setup-wizard to app

* fix: only re_route if not on setup-wizard.

* fix: remove hardcoded slide number

---------

Co-authored-by: Sagar Sharma <sagarsharma.s312@gmail.com>
(cherry picked from commit e6e59cb)
ankush pushed a commit that referenced this pull request May 31, 2023
* fix(ux): re routes based on setup_complete

If Setup Wizard is not complete re_route any route to setup_wizard and if it complete re_route setup-wizard to app

* fix: only re_route if not on setup-wizard.

* fix: remove hardcoded slide number

---------

Co-authored-by: Sagar Sharma <sagarsharma.s312@gmail.com>
(cherry picked from commit e6e59cb)

Co-authored-by: Maharshi Patel <39730881+maharshivpatel@users.noreply.github.com>
frappe-pr-bot pushed a commit that referenced this pull request Jun 6, 2023
# [14.38.0](v14.37.1...v14.38.0) (2023-06-06)

### Bug Fixes

* changed for section with features and testimonial web temlates ([9bc5c37](9bc5c37))
* changed section with card url field to small text ([1815103](1815103))
* Check if signature already exist ([6463dab](6463dab))
* currency formatter got incorrect format when use precision 0  ([#21239](#21239)) ([#21240](#21240)) ([6e5248a](6e5248a))
* declare the function ([#21261](#21261)) ([#21262](#21262)) ([9a68696](9a68696))
* do not render custom cards if workspace does not contain content ([795c894](795c894))
* doctype name localization ([#21197](#21197)) ([#21200](#21200)) ([62f25f7](62f25f7))
* fit and recenter map when section is expanded ([5d2a3ab](5d2a3ab))
* form tour set parent tab active. ([#21163](#21163)) ([#21184](#21184)) ([ae09cf9](ae09cf9))
* handle multiple webform for same doctype ([5243143](5243143))
* handle read only property ([a0e2c28](a0e2c28))
* Load map libraries at build time to avoid async issues during geolocation render ([d287dc9](d287dc9))
* Load map libraries at build time to avoid async issues during geolocation render ([e128c9d](e128c9d))
* **role:** Set desk properties (e.g. search_bar) to 1 for roles with desk access ([#21162](#21162)) ([#21231](#21231)) ([23ce3a0](23ce3a0))
* **safe_eval:** Normalize code passed before validating the code ([e58b721](e58b721))
* ui tour popover when outside viewport ([#21164](#21164)) ([#21205](#21205)) ([c3dc409](c3dc409))
* **UX:** Notify if newly created user has no roles ([#21251](#21251)) ([#21254](#21254)) ([e203bc1](e203bc1))
* **ux:** re routes based on setup_complete ([#21118](#21118)) ([#21185](#21185)) ([5e425e2](5e425e2))

### Features

* formatter for `Attach` and `Attach Image` fields ([92659ca](92659ca))
* log all DDL queries ([#21107](#21107)) ([a693c6d](a693c6d))
* read-only geolocation (GDE-86) ([bf24d87](bf24d87))
* used cached version of document in mapper ([#21186](#21186)) ([#21189](#21189)) ([7f46640](7f46640))
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 15, 2023
@maharshivpatel maharshivpatel deleted the fix-setup-wizard-ux branch December 27, 2023 08:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport version-14-hotfix backport to version 14
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0