Skip to content

Conversation

@jacobtylerwalls
Copy link
Member

@jacobtylerwalls jacobtylerwalls commented Dec 9, 2025

Closes #136

  • Update tests for permission fixes in core

force_admin=True is a development shortcut useful in tests
to avoid setting up a user with resource editor permissions
and attaching it to a request. Without a user one way or the
other, the default is the anonymous user, which recently got
its nodegroups for editing permissions revoked.

Refs archesproject/arches@1c2f647

  • Update tests for date format change in core

Refs archesproject/arches#12480


Not sure if y'all want this targeting 1.0 or 1.1?

force_admin=True is a development shortcut useful in tests
to avoid setting up a user with resource editor permissions
and attaching it to a request. Without a user one way or the
other, the default is the anonymous user, which recently got
its nodegroups for editing permissions revoked.

Refs archesproject/arches@1c2f647
@jacobtylerwalls jacobtylerwalls marked this pull request as ready for review December 9, 2025 02:21
@jacobtylerwalls jacobtylerwalls mentioned this pull request Dec 9, 2025
@johnatawnclementawn
Copy link
Member

johnatawnclementawn commented Dec 9, 2025

Thanks for figuring this out @jacobtylerwalls! Targeting dev/1.0.x would be great, 1.1 is only behind 1.0 at the moment and I was planning on pulling up the latest bugfixes (including this) from 1.0->1.1 soon (edit: which I see you've already opened the PR for 😉)

@johnatawnclementawn johnatawnclementawn changed the base branch from dev/1.1.x to dev/1.0.x December 9, 2025 17:28
@jacobtylerwalls jacobtylerwalls merged commit eb77bca into dev/1.0.x Dec 9, 2025
7 of 8 checks passed
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.

Tests are failing on main

3 participants