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

Test(Component Date Editor) #4890

Merged

Conversation

lomamech
Copy link
Collaborator

@lomamech lomamech commented Sep 4, 2020

  • Addition of a new parameter to solve the problem of the date editor
    component for forms whose .header-image is not visible by adding a new
    component
  • Use bhDateEdit in Off Days E2E test

closes #4889

- Addition of a new parameter to solve the problem of the date editor
component for forms whose .header-image is not visible by adding a new
component
- Use bhDateEdit in Off Days E2E test

closes Third-Culture-Software#4889
Copy link
Collaborator

@jeremielodi jeremielodi left a comment

Choose a reason for hiding this comment

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

Good job @lomamech, thanks

@jeremielodi
Copy link
Collaborator

bors r+

bors bot added a commit that referenced this pull request Sep 8, 2020
4890: Test(Component Date Editor) r=jeremielodi a=lomamech

- Addition of a new parameter to solve the problem of the date editor
component for forms whose .header-image is not visible by adding a new
component
- Use bhDateEdit in Off Days E2E test

closes #4889

Co-authored-by: Chris Lomame <lomamech@gmail.com>
@bors
Copy link
Contributor

bors bot commented Sep 8, 2020

Build failed:

@jeremielodi
Copy link
Collaborator

bors try

bors bot added a commit that referenced this pull request Sep 8, 2020
@bors
Copy link
Contributor

bors bot commented Sep 8, 2020

try

Build failed:

@jniles
Copy link
Collaborator

jniles commented Sep 9, 2020

bors try

bors bot added a commit that referenced this pull request Sep 9, 2020
@jeremielodi
Copy link
Collaborator

@jniles I solved this problem(bhDateEditor bug) in #4837
Don't worry if this fails

@jniles
Copy link
Collaborator

jniles commented Sep 9, 2020

Oh! Can this PR be closed then?

@jeremielodi
Copy link
Collaborator

Not yet, I just arranged the dateEditor ent-to-end test bug

@bors
Copy link
Contributor

bors bot commented Sep 9, 2020

@jeremielodi
Copy link
Collaborator

@jniles you can merge if the bors passes successfully

@jniles
Copy link
Collaborator

jniles commented Sep 9, 2020

bors r+

@bors
Copy link
Contributor

bors bot commented Sep 9, 2020

@bors bors bot merged commit 781a922 into Third-Culture-Software:master Sep 9, 2020
@lomamech lomamech deleted the bugComponentDateEditor branch October 2, 2020 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problematic for the E2E test of the bh-date-editor component
4 participants