Skip to content

Conversation

@Artistic18
Copy link

@Artistic18 Artistic18 commented Feb 26, 2020

Fixes #4030

Short description of what this resolves:

Changes proposed in this pull request:

Checklist

  • [ x] I have read the Contribution & Best practices Guide.
  • [ x] My branch is up-to-date with the Upstream development branch.
  • [ x] The acceptance, integration, unit tests and linter pass locally with my changes
  • [ x] I have added tests that prove my fix is effective or that my feature works
  • [ x] I have added necessary documentation (if appropriate)

@vercel
Copy link

vercel bot commented Feb 26, 2020

This pull request is being automatically deployed with ZEIT Now (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://zeit.co/eventyay/open-event-frontend/7bqg9eh1a
✅ Preview: https://open-event-frontend-git-fork-artistic18-development.eventyay.now.sh

@codecov
Copy link

codecov bot commented Feb 26, 2020

Codecov Report

Merging #4150 into development will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           development    #4150   +/-   ##
============================================
  Coverage        22.06%   22.06%           
============================================
  Files              461      461           
  Lines             4749     4749           
  Branches             2        2           
============================================
  Hits              1048     1048           
  Misses            3700     3700           
  Partials             1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9b0a9f9...4cd7636. Read the comment docs.

Copy link
Member

@snitin315 snitin315 left a comment

Choose a reason for hiding this comment

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

Solve the whole issue in one PR.

@Artistic18
Copy link
Author

Can you please tell which file should I make the changes.

@snitin315
Copy link
Member

open-event-frontend/app/templates/components/forms/orders/order-form.hbs

@kushthedude
Copy link
Member

Stale!

@kushthedude kushthedude closed this Mar 8, 2020
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.

Billing Info form: FE check not working for Company field

3 participants