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

IBX-8135: Remove hautelook/templated-uri-bundle fork #88

Merged
merged 1 commit into from
Jun 6, 2024

Conversation

glye
Copy link
Contributor

@glye glye commented Apr 23, 2024

🎫 Issue IBX-8135

Related PRs:

ibexa/oss#151
https://github.com/ibexa/dashboard/pull/124
ibexa/system-info#51

Description:

The Ibexa fork of hautelook/templated-uri-bundle is not needed anymore, we can use the original.

Unsure if we should bump to 4.0.

Tests should pass once ibexa/oss#151 is merged.

For QA:

Sanities only.

@glye glye force-pushed the ibx8135_remove_templated_uri_bundle branch from 1ea7ece to 3439b54 Compare April 24, 2024 08:33
@glye glye force-pushed the ibx8135_remove_templated_uri_bundle branch 2 times, most recently from b954697 to 7568132 Compare April 24, 2024 16:29
@glye glye force-pushed the ibx8135_remove_templated_uri_bundle branch 3 times, most recently from cb6396b to bdcdcf5 Compare May 10, 2024 08:51
The Ibexa fork of hautelook/templated-uri-bundle is not needed anymore, we can use the original.
@adamwojs adamwojs force-pushed the ibx8135_remove_templated_uri_bundle branch from bdcdcf5 to debd18b Compare June 6, 2024 04:59
Copy link

sonarqubecloud bot commented Jun 6, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@adamwojs
Copy link
Member

adamwojs commented Jun 6, 2024

We need wait for #98 merge

@adamwojs adamwojs merged commit ff31789 into main Jun 6, 2024
8 of 9 checks passed
@adamwojs adamwojs deleted the ibx8135_remove_templated_uri_bundle branch June 6, 2024 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants