Skip to content

Conversation

@backspace
Copy link
Owner

@backspace backspace commented Sep 22, 2025

This closes #184.

  • decide between toasts/alerts
  • cancellation dropdown not rendering/visible
  • move selectChoose inside page objects?
  • figure out missing icons (food, car?)
  • manual name entry with no matching ride doesn’t show
  • query parameters for ride filtering are broken (predates this PR but)
  • user calendar UI is weird
  • cannot open modals more than once?
  • cancellation dropdown not setting value
  • ride start/end blankness validation is swallowed
  • decide on text for non-match visitor name submission explanation
  • in “an error when an admin tries to create a commitment is displayed”, is commitment still displayed after save failure?
  • check everything on mobile

Mysterious console errors about decorators? But it works…
sometimes.
@github-actions
Copy link

github-actions bot commented Sep 22, 2025

Test Results

112 tests  +6   111 ✅ +6   25s ⏱️ - 1m 2s
  1 suites ±0     1 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit 6566f57. ± Comparison against base commit ff403ce.

This pull request removes 2 and adds 8 tests. Note that renamed tests count towards both.
Chrome 140.0 ‑ Acceptance | login: a failed login shows an error
Chrome 140.0 ‑ Acceptance | reports: it lists gas prices and reïmbursement rates
Chrome 140.0 ‑ Acceptance | forgot: a reset email failure shows an alert until it succeeds
Chrome 140.0 ‑ Acceptance | gas prices: it lists gas prices and reïmbursement rates
Chrome 140.0 ‑ Acceptance | login: a failed login shows an error that clears on retry
Chrome 140.0 ‑ Acceptance | people: people can be sorted by name and last ride
Chrome 140.0 ‑ Acceptance | rides: a visitor name can be entered manually even when matches exist
Chrome 140.0 ‑ Acceptance | rides: timespan validation errors are shown when manual overrides are not used
Chrome 140.0 ‑ Acceptance | sidebar: a badge is shown on the toggle when notifications exist and the sidebar is closed
Chrome 140.0 ‑ Acceptance | sidebar: the header toggle opens and closes the sidebar

♻️ This comment has been updated with latest results.

# Conflicts:
#	package-lock.json
I couldn’t get the application to run anymore, maybe something
about pnpm…? ugh. But now I can replace components one at a time.
This makes the application render at least, very little to
show since all the Ember Paper components are empty, but
without this there was an `decorator is not a function` error
that would abort rendering.
it’s a start!
There’s a focus trap problem with the modal, which is a
major aspect of this, something to continue on.
It seems like it might have been working but 1.1.1 says
it needs Ember 4.16 which I’m not on.
This has rendering weirdness and I couldn’t figure out how
to get page objects to fit the structure 😞
This will need further cleanup.
whew! Another failing test but mostly working
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.

Update to Helios?

2 participants