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

Refactor: Remove logoClickConfirm and make user go to the experiment dashboard directly #1303

Merged
merged 2 commits into from
Oct 2, 2024

Conversation

drikusroor
Copy link
Contributor

@drikusroor drikusroor commented Oct 1, 2024

Resolves #1301 Resolves #1296

I seem to remember having overheard this issue was discovered on production and would ideally be fixed before Thursday (2024-10-03), hence the PR to main as a hotfix.

@drikusroor drikusroor self-assigned this Oct 1, 2024
Copy link
Collaborator

@BeritJanssen BeritJanssen left a comment

Choose a reason for hiding this comment

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

You overheard correctly! Thanks for already picking this up! I was about to start on it. Can you also merge this to develop? And create a release? Thanks!

@drikusroor drikusroor merged commit f538ebb into main Oct 2, 2024
7 checks passed
@drikusroor drikusroor deleted the feat/1301-header-logo-direct-link-main branch October 2, 2024 12:52
drikusroor added a commit that referenced this pull request Nov 5, 2024
* fix: correct video links

* fix: Refactor Docker commands in scripts (#1282)

Update the Docker commands in the scripts to use the "docker compose" syntax instead of "docker-compose".

* fix: Fix some search & replace too many, where I mistook `docker-compose-deploy.yml` for a `docker-compose` command (#1283)

* fix(style): Prevent percentage circle from overlapping background circle (#1281)

* Added: Disable pinch zoom on iOS (#1284)

* feat: Add useDisableIOSPinchZoomOnTouchDevices hook

This commit adds a new hook called useDisableIOSPinchZoomOnTouchDevices to the frontend codebase. The hook is responsible for preventing pinch zoom gestures on iOS devices. It is used in the App component to disable pinch zoom on touch devices.

* feat: Add touch-action property to disable pinch zoom on iOS

* fix(style): Move percentage circle down 100% to align with background circle

* fix: use vimeo for videos

* fix: redefine ButtonArray button gaps

* fix: docker compose without hyphen

* fix: update podman-compose file names (#1290)

* adjust debrief template

* Fixed: Fix Circle percentage overlap issue (attempt 3) (#1291)

* style: Move rotate transform and transform origin to the circle's svg level

* story(AutoPlay): Add `AutoPlay` story

* chore(deps): Update `package.json` version to `2.2.5` (#1293)

* fix: remove non-breaking spaces when showing composer / ensemble / piece

* fix: adjust scaling of image in th_preverbal

* chore: update version

* chore: remove unused import

* Refactor: Remove `logoClickConfirm` and make user go to the experiment dashboard directly (#1303)

* refactor: Remove `logoClickConfirm` and make user go to the experiment dashboard directly

* chore(deps): Update package version to 2.2.7

---------

Co-authored-by: BeritJanssen <berit.janssen@gmail.com>
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.

2 participants