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

url function around static resource links #101

Closed
wants to merge 1 commit into from

Conversation

Miniontoby
Copy link
Contributor

Wrapped the static resource links in the url function to make it sync to APP_URL

This would allow later on to support running in an subfolder

Wrapped the static resource links in the url function to make it sync to APP_URL
This would allow later on to support running in an subfolder
Copy link
Contributor

github-actions bot commented Apr 18, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Miniontoby
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@iamkubi
Copy link
Contributor

iamkubi commented Apr 18, 2024

Hmm, looking at the CLA failure. It doesn't look like it added your signature or rechecked after your comment, so something probably broke.

@Miniontoby
Copy link
Contributor Author

recheck

@Miniontoby
Copy link
Contributor Author

Miniontoby commented Apr 18, 2024

Maybe it will trigger now?

Update: yes it has

@iamkubi
Copy link
Contributor

iamkubi commented Apr 18, 2024

I think I likely broke the CLA workflow for this PR by messing with the signatures in the middle. Safe to ignore if it continues to fail for this PR.

@Miniontoby
Copy link
Contributor Author

The check just needs to be retriggered/retried/reexecuted. I know that that is a thing with github actions

@lancepioch lancepioch changed the base branch from 3.x to master April 18, 2024 21:03
@lancepioch
Copy link
Member

See #102

@lancepioch lancepioch closed this Apr 18, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Apr 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants