Skip to content

BTI-1111 fix: change hardcoded wp-admin url#489

Merged
vildanbina merged 2 commits into
buckaroo-it:developfrom
omnisite:fix-wp-admin-url
Jun 25, 2026
Merged

BTI-1111 fix: change hardcoded wp-admin url#489
vildanbina merged 2 commits into
buckaroo-it:developfrom
omnisite:fix-wp-admin-url

Conversation

@omnisite

Copy link
Copy Markdown

The hardcoded wp-admin URL gives a 404 in setups where wp-admin is renamed or in another location (like /wp/wp-admin/ in Bedrock setups). The current buckaroo_global.ajax_url is set to home('/') so that is why I added "admin_ajax_url" on "buckaroo_global". You should probably do your own npm run build (there was no .nvmrc so I used node 20).

@Buckaroo-Rene Buckaroo-Rene self-assigned this Jun 19, 2026
@Buckaroo-Rene Buckaroo-Rene changed the title fix: change hardcoded wp-admin url BTI-1111 fix: change hardcoded wp-admin url Jun 19, 2026
@Buckaroo-Rene

Copy link
Copy Markdown
Contributor

Hi @omnisite ,

Thank you for informing us about the issue you’ve encountered.
Our team will look into this and investigate a suitable solution.
For reference, we’ve created an internal ticket: BTI-1111.

@Buckaroo-Rene Buckaroo-Rene added Status: Backlog This is currently on our backlog and not scheduled yet. Priority: Normal This issue has a normal/medium priority. labels Jun 19, 2026
@vildanbina

Copy link
Copy Markdown
Collaborator

@omnisite will you fix the conflicts?

@omnisite

omnisite commented Jun 24, 2026

Copy link
Copy Markdown
Author

@vildanbina these are in the dist files. That is why I commented in my PR that it would be best if you do the build and then commit the dist files. I can also do it, but there is no nvmrc file, so I do not know which version node must be used.

# Conflicts:
#	assets/js/dist/blocks.asset.php
#	assets/js/dist/blocks.js
@vildanbina

Copy link
Copy Markdown
Collaborator

i merged develop and rebuilt the dist, conflicts are gone now.

about the node version: there's no .nvmrc but package-lock.json expects node 20. i built on a newer node and the output matched develop's other bundles exactly, so it's fine.

@vildanbina vildanbina merged commit 7c76b92 into buckaroo-it:develop Jun 25, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Priority: Normal This issue has a normal/medium priority. Status: Backlog This is currently on our backlog and not scheduled yet.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants