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

Refactor Join Steps #840

Open
wants to merge 7 commits into
base: develop
Choose a base branch
from
Open

Refactor Join Steps #840

wants to merge 7 commits into from

Conversation

vinit717
Copy link
Member

@vinit717 vinit717 commented Jan 18, 2024

Date: 18-01-2024

Developer Name: vinit


Issue Ticket Number:-

Description:

In this pr join steps refactoration is done including submit btn to disable if previous any filled is not filled

Is Under Feature Flag

  • Yes
  • No

Database changes

  • Yes
  • No

Breaking changes (If your feature is breaking/missing something please mention pending tickets)

  • Yes
  • No

Is Development Tested?

  • Yes
  • No

Tested in staging?

  • Yes
  • No

Add relevant Screenshot below ( e.g test coverage etc. )

image
image

Follow-up issue

Copy link

cloudflare-workers-and-pages bot commented Jan 18, 2024

Deploying with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9373d1c
Status: ✅  Deploy successful!
Preview URL: https://1f326003.www-rds.pages.dev
Branch Preview URL: https://refactor-join-steps.www-rds.pages.dev

View logs

@@ -15,7 +15,7 @@
{{#if this.errorMessage.whyRds}}
<div class='error__message'>{{this.errorMessage.whyRds}}</div>
{{/if}}

<div class="input-box__hour">
Copy link
Member

Choose a reason for hiding this comment

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

Let's make the class more generic

Suggested change
<div class="input-box__hour">
<div class="input-box__time">

assert.dom('[data-test-onboarding-card-modal]').exists();
assert.dom('[data-test-error-message]').doesNotExist();
Copy link
Member

Choose a reason for hiding this comment

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

is the test for exist case already written?

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.

3 participants