Skip to content

FF: fix startView across OS platforms that opens psychopyApp.py#6869

Merged
TEParsons merged 1 commit into
psychopy:releasefrom
mh105:release-startview-psychopyApp
Sep 26, 2024
Merged

FF: fix startView across OS platforms that opens psychopyApp.py#6869
TEParsons merged 1 commit into
psychopy:releasefrom
mh105:release-startview-psychopyApp

Conversation

@mh105
Copy link
Copy Markdown
Contributor

@mh105 mh105 commented Sep 23, 2024

Same fix as #6789. I found out today that this issue of automatically opening psychopyApp.py is happening on Windows standalone app as well. I'm not sure what is trying to open psychopyApp.py when launching the app, but this FF commit can at least make the startView setting in preference work correctly and not always open Coder view due to the psychopyApp.py.

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 23, 2024

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 49.62%. Comparing base (713ea1f) to head (f971e34).
Report is 18 commits behind head on release.

Additional details and impacted files
@@           Coverage Diff            @@
##           release    #6869   +/-   ##
========================================
  Coverage    49.62%   49.62%           
========================================
  Files          332      332           
  Lines        61201    61206    +5     
========================================
+ Hits         30373    30376    +3     
- Misses       30828    30830    +2     
Components Coverage Δ
app ∅ <ø> (∅)
boilerplate ∅ <ø> (∅)
library ∅ <ø> (∅)
vm-safe library ∅ <ø> (∅)

@TEParsons
Copy link
Copy Markdown
Contributor

I've just realised this is the same error I was fixing in #6875... The two solutions don't clash though, so I'll pull this in too as a "belt and braces" approach :)

@TEParsons TEParsons merged commit 624fbb1 into psychopy:release Sep 26, 2024
@mh105 mh105 deleted the release-startview-psychopyApp branch October 29, 2024 16:02
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