Skip to content

Supported font formats mismatch#38465

Merged
wilsonge merged 1 commit intojoomla:4.2-devfrom
brianteeman:supported_font_formats
Sep 6, 2022
Merged

Supported font formats mismatch#38465
wilsonge merged 1 commit intojoomla:4.2-devfrom
brianteeman:supported_font_formats

Conversation

@brianteeman
Copy link
Copy Markdown
Contributor

The default values in the xml include woff2
The default value in the db does not include woff2

The default values on a new install should match those in the component xml and this PR corrects that.

We can't fix it on update as that would be changing user data.

To test either code review or do a clean install from the provided build

The default values in the xml include woff2
The default value in the db does not include woff2

The default values on a new install should match those in the component xml and this PR corrects that.

We can't fix it on update as that would be changing user data.

To test either code review or do a clean install from the provided build
@Abernyte-Git
Copy link
Copy Markdown

I have tested this item ✅ successfully on 428c630


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38465.

1 similar comment
@viocassel
Copy link
Copy Markdown
Contributor

I have tested this item ✅ successfully on 428c630


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38465.

@Quy Quy added the RTC This Pull Request is Ready To Commit label Aug 15, 2022
@Quy
Copy link
Copy Markdown
Contributor

Quy commented Aug 15, 2022

RTC


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/38465.

@richard67 richard67 added the Maintainers Checked Used if the PR is conceptional useful label Aug 18, 2022
@wilsonge wilsonge merged commit 05d919e into joomla:4.2-dev Sep 6, 2022
@wilsonge
Copy link
Copy Markdown
Contributor

wilsonge commented Sep 6, 2022

Thanks!

@joomla-cms-bot joomla-cms-bot removed the RTC This Pull Request is Ready To Commit label Sep 6, 2022
@brianteeman brianteeman deleted the supported_font_formats branch December 25, 2022 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Maintainers Checked Used if the PR is conceptional useful

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants