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

Update welcome/index.html spec #1009

Closed
christinach opened this issue Feb 26, 2024 · 1 comment
Closed

Update welcome/index.html spec #1009

christinach opened this issue Feb 26, 2024 · 1 comment
Assignees

Comments

@christinach
Copy link
Member

christinach commented Feb 26, 2024

run

bundle exec rspec ./spec/views/welcome/index.html.erb_spec.rb

Update the lux component prefix that is missing.
Make the test pass.

Checkout a new branch from PR#984

Open the PR against PR#984

@maxkadel maxkadel self-assigned this Feb 27, 2024
maxkadel added a commit that referenced this issue Feb 27, 2024
sandbergja pushed a commit that referenced this issue Feb 27, 2024
christinach pushed a commit that referenced this issue Feb 29, 2024
@christinach
Copy link
Member Author

The specs are now passing.

christinach pushed a commit that referenced this issue Mar 8, 2024
christinach pushed a commit that referenced this issue Mar 8, 2024
christinach pushed a commit that referenced this issue Mar 13, 2024
christinach pushed a commit that referenced this issue Mar 15, 2024
christinach pushed a commit that referenced this issue Mar 15, 2024
christinach pushed a commit that referenced this issue Mar 27, 2024
kevinreiss added a commit that referenced this issue Apr 1, 2024
* Use lux-styleguidist
Update vue and vue dependencies to vue3

Co-authored-by: Jane Sandberg <[email protected]>

* Remove vue@2 dependencies from tests

Co-authored-by: Jane Sandberg <[email protected]>

* upgrades vue-test-utils to a compatible version and removes localVue refereces from the test files

Co-authored-by: Christina Chortaria <[email protected]>
Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Kevin Reiss <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>
Co-authored-by: Shaun Ellis <[email protected]>

* Replace instances of InputText with LuxInputText

Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>
Co-authored-by: Shaun Ellis <[email protected]>

* In vue 3, we no longer need .$el

* Use the Vue 3 plugin and component loading syntax (#994)

Also, use the esm-bundler version of vue, which resolved a console warning

* Use lux-heading and lux-text-style components from lux (#996)

* These components now have the lux- prefix
* They need to be added to each instance of the vue app

Co-authored-by: Christina Chortaria <[email protected]>

* Add Rails UJS back to the javascript (#997)

It previously was giving me a console warning when it was included; I guess that's resolved now, and we need it in order to log in to CAS

* Use lux-data-table, rather than data-table (#1005)

* Build and use lux-styleguidist in CI (#1007)

Start by cloning into ~/lux-styleguidist and adding yalc

Try building lux-styleguidist

NPM install first

Add lux-styleguidist to approvals using yalc

Add lux-styleguidist to approvals using yalc

* Fix indentation using yarn lint (#1008)

* Update hyperlink to lux-hyperlink

Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Kevin Reiss <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>

* Update lux to 5.0.0-alpha.12

Co-authored-by: Beck Davis <[email protected]>
Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>

* Add prefix in Lux Components

Co-authored-by: Beck Davis <[email protected]>
Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>

* Vue components call lux components with the lux- prefix (#1022)

* Eslint should ignore the Rails/ujs import (#1023)

* Use the LuxInputSelect's focusSelect() method (#1026)

Also, update components to use the lux- prefix

Co-authored-by: Christina Chortaria <[email protected]>

* Resolve yarn test warnings

* Update eslint ecmaVersion (#1027)

ESlint was unable to parse optional chaining, since it was using 2018 rules.
Also, the line it was complaining about had an unused assignment, let's remove it!

* Fix views/shared/ specs (#1030)

Closes #1011

* Fix views/absence_requests specs (#1031)

Closes #1012

* Fixes #1009 (#1028)

* Fix test failures for spec/views/travel_requests (#1029)

* i1013: address delegate system test failure (#1032)

* Update components with prefixs. Towards #1017 (#1033)

* Update components with prefixs. Towards #1017 (#1034)

* Update components with lux prefix towards #1017 (#1035)

* Update dropdown prefix with lux (#1037)

* Upgrading absence request form for new lux (#1038)

* Makes progress toward i1013
* Installs the latest alpha release of lux-styleguidist, so that the
  datepicker component is registered

* Update components with lux prefix: my_requests_spec, edit.html.erb, new.html.erb (#1039)

* Use lux-tag and lux-input-button (#1041)

Co-authored-by: Jane Sandberg <[email protected]>

* Get a little farther in the test before failing (#1044)

The date range is not currently actually being filled in. You can verify this by turning off "headless" mode and running the test with a byebug after line 64. There are two main errors in the console:
    `TypeError: m.includes is not a function`, raised from the updateRangeInput function in lux-styleguidist
    `TypeError: Cannot read properties of undefined`, raised from the toLocaleDateString function in lux-styleguidist
    See pulibrary/lux-design-system#107

* Update lux version to 5.0.0-alpha.15 (#1045)

Co-authored-by: Jane Sandberg <[email protected]>

* Update selectors in view tests to match new lux components (#1046)

* Update lux to alpha.16 (#1052)

Co-authored-by: Kim Leaman <[email protected]>
Co-authored-by: Beck Davis <[email protected]>
Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Kevin Reiss <[email protected]>
Co-authored-by: Max Kadel <[email protected]>
Co-authored-by: regineheberlein <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>

* lux 5.0.0-alpha.17 (#1056)

* Run CI against published lux-styleguidist release (#1057)

* Lux 5.0.0-alpha18 (#1058)

* Mount the vue application to the parent of our top component (#1059)

Vue 3 needs all vue components to be the descendants of
the element that we mount the application to.  In Vue 2,
we could mount to a component element.

This moves the mount point to the lux-library-header's parent
component, <body>, so that Vue can render the header.

See https://v3-migration.vuejs.org/breaking-changes/mount-changes.html

* Listen for inputvaluechange event, rather than the input event (#1061)

The input event has changed in lux-styleguidist, since it
causes problems with the datepicker.

* update lux to 5.0.0-alpha.19 (#1060)

* Update lux to alpha.21 (#1064)

* Fix styles in the help page (#1072)

* lux alpha.22 release (#1073)

Co-authored-by: Jane Sandberg <[email protected]>

* Add lux prefix in various components (#1074)

* Update wrapper component with lux prefix (#1075)

* Update alert with lux prefix- fix delegation (#1076)

* Lux prelease alpha.23 (#1077)

* Lux prerelease alpha.25 (#1087)

* Lux 5.0.0-alpha.26 prerelease (#1089)

* Update slot syntax (#1094)

* Upgrade lux to 5.0.0-alpha.27 (#1103)

* Fix test for card-header (#1107)

Closes #1099

* Add lux prefix in help page (#1111)

* Removed hard-coded styles (#1112)

* Removed hard-coded styles

* Fixed missed merge conflict

---------

Co-authored-by: Ryan Laddusaw <[email protected]>

* Sending the escape key to close the autocomplete that was blocking buttons (#1113)

Co-authored-by: Ryan Laddusaw <[email protected]>

* Update lux to 5.0.0-alpha.28 (#1114)

* Sending tab to clear autocomplete instead of escape (#1115)

* Sending the escape key to close the autocomplete that was blocking buttons

* Changed escape to tab

---------

Co-authored-by: Ryan Laddusaw <[email protected]>

* lux release 5.0.0-alpha.29 (#1119)

* Update lux to alpha-5.0.0-30 (#1120)

* Update lux to alpha-5.0.0-31 (#1122)

Fixes luxInputText border #1121
Fixes bug in autocompleteInput

* update lux to 5.0.0-alpha.32 (#1124)

* Update lux to alpha-5.0.0-33 release (#1125)

Co-authored-by: Jane Sandberg <[email protected]>

* Lux 5.0.0-beta.2 (#1128)

Mark Absence workflow tests as pending

Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Kevin Reiss <[email protected]>

* lux-design-system release 5.0.0-beta.3 (#1132)

* Update location_loader_spec Array spec with contain_exactly (#1133)

Co-authored-by: Jane Sandberg <[email protected]>

---------

Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Shaun Ellis <[email protected]>
Co-authored-by: Christina Chortaria <[email protected]>
Co-authored-by: Kevin Reiss <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>
Co-authored-by: Shaun Ellis <[email protected]>
Co-authored-by: Jane Sandberg <[email protected]>
Co-authored-by: Max Kadel <[email protected]>
Co-authored-by: Beck Davis <[email protected]>
Co-authored-by: Kim Leaman <[email protected]>
Co-authored-by: Max Kadel <[email protected]>
Co-authored-by: regineheberlein <[email protected]>
Co-authored-by: rladdusaw <[email protected]>
Co-authored-by: Ryan Laddusaw <[email protected]>
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

No branches or pull requests

2 participants