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

subtitle in form editor #1662

Merged
merged 1 commit into from
Nov 26, 2019
Merged

subtitle in form editor #1662

merged 1 commit into from
Nov 26, 2019

Conversation

alanlong9278
Copy link
Contributor

Description

subtitle in form editor

If this is a bug fix, please describe the root cause and analysis of this problem.

Task Item

Please include a link to the related issue. Ex. Closes #<issue #>

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Code refactor (non-breaking change which improve code quality, clean up, add tests, etc)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Doc update (document update)

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have functionally tested my change

Screenshots

Please include screenshots or gifs if your PR include UX changes.

@alanlong9278 alanlong9278 merged commit ff5e1af into zeye/design/trigger Nov 26, 2019
@alanlong9278 alanlong9278 deleted the julong/trigger branch November 26, 2019 07:23
boydc2014 pushed a commit that referenced this pull request Nov 29, 2019
* new trigger

* add `subtitle` in ConceptLabels to override some title

* refine new css of Trigger

* fix an unsafe prop visit

* fix eslint import path

* move styles to 'triggerStyles.ts'

* use lodash.get in Trigger.tsx

* add subtitle in form editor (#1662)

* Final changes

* Remove unnecessary guard

* enable debug when running tests

* fix ci: BreadCrumb

* fix ci: CreateNewBot

* revert BreadCrumb

* delete data.json before each e2e test

* More labeling

* Fix e2e test

* fix visual designer spec

* use findAllBy for breadcrumb spec

sometimes more than one breadcrumb is found. this will use the last one found

* use debug logger

* pipe server output to log file and print after tests are run

* fix up e2e script

* run headless
* pipe server logs to file
* only invoke cleanup function once

* fix visual designer spec again

* display server logs

* add more debug logging

* always print e2e logs

* Base leileizh's e2e fixes

* update the node env
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.

1 participant