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

[CMSP-1140] fix subdirectory paths #142

Closed
wants to merge 122 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
122 commits
Select commit Hold shift + click to select a range
0b0bd54
abstract out the site_id and pass a site url to playwright
jazzsequence Jul 11, 2024
0582ff5
allow site name to come from environment
jazzsequence Jul 11, 2024
b4cad69
move playwright tests to repeatable composate action
jazzsequence Jul 11, 2024
fbd03cb
run actions/checkout first
jazzsequence Jul 11, 2024
e2d5289
remove checkout from composite action
jazzsequence Jul 11, 2024
fc3fe33
Update playwright.yml to use the correct path for run-playwright-tests
jazzsequence Jul 11, 2024
61ec424
pass all the parameters from the workflow that has that info
jazzsequence Jul 11, 2024
f6d3a1f
add shell
jazzsequence Jul 11, 2024
84c1a92
add logging
jazzsequence Jul 11, 2024
ffdf563
log the deletion, too
jazzsequence Jul 11, 2024
8d8b177
pass the site name to the playwright tests
jazzsequence Jul 11, 2024
63e00b5
fix broken colors
jazzsequence Jul 11, 2024
5b69e83
add stdbuf to commands to log everything to the terminal
jazzsequence Jul 11, 2024
cebc4ba
shorten the names
jazzsequence Jul 11, 2024
d5c3ce7
set the upstream id
jazzsequence Jul 11, 2024
2e3dda9
reference the action file directly
jazzsequence Jul 11, 2024
b46e70f
create application.php files for subdir & subdom multisite
jazzsequence Jul 11, 2024
8f16228
copy the config file to the config/ directory
jazzsequence Jul 11, 2024
1ac3b60
absolutely do not use the action.yml path
jazzsequence Jul 11, 2024
a4c616d
fix linting in two new files
jazzsequence Jul 11, 2024
c309791
just use actual upstream IDs
jazzsequence Jul 11, 2024
2253836
remove comment
jazzsequence Jul 11, 2024
618dd97
don't set the ID to the new ID
jazzsequence Jul 11, 2024
333b7e7
check the multisite install when we status check
jazzsequence Jul 11, 2024
d46e7a1
fix broken conditional
jazzsequence Jul 12, 2024
02774a1
the conditional needs to wrap around the whole multisite block
jazzsequence Jul 12, 2024
438ff24
add a db reset before site install
jazzsequence Jul 12, 2024
425c4b2
add the updated application.php file before the multisite check
jazzsequence Jul 12, 2024
32ad6ce
make sure we're in the right directory
jazzsequence Jul 12, 2024
d5b31fd
rename the file
jazzsequence Jul 12, 2024
b033c64
be specific about the type of multisite in the commit message
jazzsequence Jul 12, 2024
c573537
move the wait to the top
jazzsequence Jul 12, 2024
1058a70
use the correct site url (not the copy pasta one)
jazzsequence Jul 12, 2024
a450b0f
display art for no good reason
jazzsequence Jul 12, 2024
1bf264e
wait for the config to hit the server
jazzsequence Jul 12, 2024
15fcd2b
yeah 🤦‍♂️
jazzsequence Jul 12, 2024
39b9847
I guess we just can't do a colored logo
jazzsequence Jul 12, 2024
b9fc854
set a max wait on the wait before installing graphql
jazzsequence Jul 12, 2024
6da6d7b
only add the subsite if it doesn't already exist
jazzsequence Jul 12, 2024
a7caf17
split the clone and copy updates apart
jazzsequence Jul 12, 2024
2706204
we do actually need the github token
jazzsequence Jul 12, 2024
d117cb5
fix typo
jazzsequence Jul 12, 2024
16bf9ef
don't fail if there's nothing to commit
jazzsequence Jul 12, 2024
1ecca27
just see if a foo site exists
jazzsequence Jul 12, 2024
9c94e79
break the composite workflow into a script
jazzsequence Jul 12, 2024
bebe3b4
PRIVATE_KEY not SSH_KEY
jazzsequence Jul 12, 2024
45ef0e2
fix terminus token, too
jazzsequence Jul 12, 2024
acdd8dc
set defaults & move functions to the bottom
jazzsequence Jul 12, 2024
de3dce1
shellcheck
jazzsequence Jul 12, 2024
ed3aed6
fix the other places inputs were broken
jazzsequence Jul 12, 2024
0d6f3b0
fix secret name
jazzsequence Jul 12, 2024
6a5c265
delete the sites when we're done
jazzsequence Jul 12, 2024
cdc8d47
remove any exits that aren't errors
jazzsequence Jul 12, 2024
99b0207
fix file copy globbing
jazzsequence Jul 12, 2024
6cbc94f
remove composite action
jazzsequence Jul 12, 2024
ae516a6
move install dependencies into action & pantheon host key into script
jazzsequence Jul 12, 2024
bc2221d
get last commit message and export to the environment
jazzsequence Jul 12, 2024
68268d3
remove host key command
jazzsequence Jul 12, 2024
e283f40
use the github token
jazzsequence Jul 12, 2024
e0d7ef0
cd to the workspace
jazzsequence Jul 12, 2024
37785dc
need to get PR number
jazzsequence Jul 12, 2024
be9b085
echo the commit message for debugging
jazzsequence Jul 12, 2024
8c6d1a2
use upstream name instead of id
jazzsequence Jul 12, 2024
b97ae4a
move playwright tests into the main workflow
jazzsequence Jul 12, 2024
4297f91
remove the blue var
jazzsequence Jul 12, 2024
64f6f03
need to install deps in single site
jazzsequence Jul 12, 2024
4cb5178
multisite doesn't add host key
jazzsequence Jul 12, 2024
c705e83
rename run-playwright-tests to setup-
jazzsequence Jul 12, 2024
d68fbb2
Commit GraphQL to the repo
jazzsequence Jul 12, 2024
0700e05
move the workflow:wait to after we copy updates
jazzsequence Jul 12, 2024
6558503
use test instead of comparing value
jazzsequence Jul 12, 2024
a49a953
don't use test, just run the command
jazzsequence Jul 12, 2024
d21363d
set connection to git always
jazzsequence Jul 12, 2024
fc1fe05
store the value of the config get in a var and check that
jazzsequence Jul 12, 2024
2776c8a
install wp before doing status check
jazzsequence Jul 12, 2024
4bc7e18
fix subdir siet url vars
jazzsequence Jul 12, 2024
d165e9f
force db reset
jazzsequence Jul 12, 2024
1873e53
the script shouldn't fail if foo site isn't found
jazzsequence Jul 12, 2024
7c9a814
fix site title
jazzsequence Jul 16, 2024
4d8cf2e
store site name and url in Github env
jazzsequence Jul 16, 2024
d98c8e6
normalize rest endpoint for multisite
jazzsequence Jul 16, 2024
f7a980d
separately apply the filter to rest urls
jazzsequence Jul 16, 2024
3e134da
filter the rest endpoint, too
jazzsequence Jul 16, 2024
07823c8
remove all the wp-json handling
jazzsequence Jul 16, 2024
2af6a6a
adjust the playwright test to expect the api endpoint to be /wp/wp-json
jazzsequence Jul 16, 2024
095c757
change the site name before we test the subsite
jazzsequence Jul 16, 2024
1cf97f6
the graphql endpoint changes if we're on a subsite, too
jazzsequence Jul 16, 2024
f2e6271
flush the cache for the sub-site if we're doing multisite
jazzsequence Jul 16, 2024
0459d7c
fix the toBeTruthy check
jazzsequence Jul 16, 2024
850dee3
echo with -e for color
jazzsequence Jul 16, 2024
c940e23
fix spacing
jazzsequence Jul 16, 2024
62ba688
only log in if not already logged in
jazzsequence Jul 16, 2024
3ccabb8
don't flush permalinks here
jazzsequence Jul 16, 2024
a9355d1
add tests for subdomain
jazzsequence Jul 16, 2024
df5f36d
fix the subdom site url
jazzsequence Jul 16, 2024
857ff51
[CMSP-1140] Move cookie settings down to use Pantheon hostname (#141)
jazzsequence Jul 17, 2024
c6c6196
wrap PANTHEON_ENVIRONMENT with an isset
jazzsequence Jul 17, 2024
ff29689
remove the comment
jazzsequence Jul 17, 2024
311a847
make the globals readonly
jazzsequence Jul 17, 2024
7ad1ae7
return early
jazzsequence Jul 17, 2024
2c6f63e
use `[[` instead of `[`
jazzsequence Jul 17, 2024
dfbe464
return early
jazzsequence Jul 17, 2024
974fa51
one terminus command to activate wp-graphql
jazzsequence Jul 17, 2024
f13d1f9
use brackets around commit msg
jazzsequence Jul 17, 2024
81d3c83
Merge branch 'cmsp-1140-add-tests-for-multisite' of github.com:panthe…
jazzsequence Jul 17, 2024
b3beeda
use printf to handle single quotes in commit messages
jazzsequence Jul 17, 2024
e9a21b2
skip creating a new variable if we do not need it
jazzsequence Jul 17, 2024
9d64a09
strip out single quotes
jazzsequence Jul 17, 2024
a16edd2
better handling of sed
jazzsequence Jul 17, 2024
de920ae
use printf
jazzsequence Jul 17, 2024
74de43c
strip newlines and carriage returns
jazzsequence Jul 17, 2024
99e3ed4
better commit message cleaning
jazzsequence Jul 18, 2024
4f1b51e
add opening slash and remove wp-admin from core paths
jazzsequence Jul 18, 2024
72f5478
remove the /wp/ handling entirely
jazzsequence Jul 18, 2024
8489d61
remove the admin and login url filters
jazzsequence Jul 18, 2024
f1270af
add phpcbf workflow
jazzsequence Jul 18, 2024
7e122a8
don't commit if there were no cbf changes
jazzsequence Jul 18, 2024
2e1bf9e
only git add php files
jazzsequence Jul 18, 2024
5abf4d7
don't say changes were detected if they weren't
jazzsequence Jul 18, 2024
a2d5717
use the right command
jazzsequence Jul 18, 2024
689c727
always pass the cbf env variable
jazzsequence Jul 18, 2024
fb551ba
Merge branch 'default' into cmsp-1140-fix-subdirectory-paths
jazzsequence Jul 18, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove checkout from composite action
jazzsequence committed Jul 11, 2024
commit e2d5289584db264688ec11dc019288bba112f900
2 changes: 0 additions & 2 deletions .github/workflows/run-playwright-tests.yml
Original file line number Diff line number Diff line change
@@ -21,8 +21,6 @@ jobs:
SITE_NAME: ${{ inputs.site_name }}

steps:
- uses: actions/checkout@v4

- name: Generate lock files
run: |
npm install --package-lock-only