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

Fix PR #346 comments #367

Merged
merged 2 commits into from
Aug 29, 2023
Merged

Fix PR #346 comments #367

merged 2 commits into from
Aug 29, 2023

Conversation

LarsSelbekk
Copy link
Member

  • feat(MatchDetail): improve error message
  • feat(MatchDetail): link branch hours in StandMatch

@LarsSelbekk LarsSelbekk added the enhancement New feature or request label Aug 25, 2023
@LarsSelbekk LarsSelbekk self-assigned this Aug 25, 2023
@vercel
Copy link

vercel bot commented Aug 25, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
bl-next ✅ Ready (Inspect) Visit Preview 💬 Add feedback Aug 25, 2023 2:19pm

@LarsSelbekk LarsSelbekk changed the title fix pr346 comments Fix PR #346 comments Aug 25, 2023
@LarsSelbekk
Copy link
Member Author

LarsSelbekk commented Aug 25, 2023

This fixes comments of #346.

Note for opening hours linking: we don't store which branch a stand match belongs to, so it's not possible to link directly to the opening hours for a specific branch without API changes. It now links to the opening hours branch selector, which if you've already chosen a branch redirects to that branch. This might be okay, but it might also be error-prone if someone has chosen the wrong branch. Also, I couldn't find a branch that actually had opening hours 🤷‍♂️

@LarsSelbekk
Copy link
Member Author

@cypress
Copy link

cypress bot commented Aug 25, 2023

4 failed tests on run #159 ↗︎

4 16 0 11 Flakiness 0

Details:

Merge 832329f into e5e89fd...
Project: bl-next Commit: 725da3c9e9 ℹ️
Status: Failed Duration: 02:50 💡
Started: Aug 25, 2023 2:24 PM Ended: Aug 25, 2023 2:27 PM
Failed  settings.spec.js • 1 failed test • UI - Electron

View Output Video

Test Artifacts
Settings page > displays correct inital elements Output Screenshots Video
Failed  forgot.spec.js • 1 failed test • UI - Electron

View Output Video

Test Artifacts
Forgot password > can navigate to and from forgot page Output Screenshots Video
Failed  login.spec.js • 1 failed test • UI - Electron

View Output Video

Test Artifacts
Login > displays all important elements Output Screenshots Video
Failed  register.spec.js • 1 failed test • UI - Electron

View Output Video

Test Artifacts
Register > displays correct initial elements Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@AdrianAndersen AdrianAndersen merged commit e908c15 into main Aug 29, 2023
7 of 11 checks passed
@AdrianAndersen AdrianAndersen deleted the fix-pr346-comments branch August 29, 2023 09:25
@AdrianAndersen AdrianAndersen removed their request for review June 17, 2024 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants