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

GCWeb: Splash page - Canada.ca assets down #2033

Open
DamonU2 opened this issue Oct 14, 2022 · 3 comments
Open

GCWeb: Splash page - Canada.ca assets down #2033

DamonU2 opened this issue Oct 14, 2022 · 3 comments

Comments

@DamonU2
Copy link

DamonU2 commented Oct 14, 2022

The working example and splash page example links both return 404, as well as the assets (at least https://wet-boew.github.io/themes-dist/GCWeb/GCWeb/assets/wmms-spl.svg and https://wet-boew.github.io/themes-dist/GCWeb/GCWeb/assets/sig-spl.svg)

@anthonyfok
Copy link
Contributor

@DamonU2 Thank you for reporting this issue! Like you, I wish GCWeb has an easier way of maintaining backward compatibility and all, but anyhow...

I took a look at recent releases, and I think I found the "culprit" in Release v11.5.0 (https://github.com/wet-boew/GCWeb/releases/tag/v11.5.0) from 29 Sept 2022:

Templates

  • Minor - Splash page - Create French version + assets cleanup
    Note: A communication about the changes is going to be sent to the departments

...

Modified files for implementation

  • ...
  • (removed) GCWeb/assets/sig-spl.png
  • (removed) GCWeb/assets/sig-spl.svg
  • (removed) GCWeb/assets/wmms-spl.png
  • (removed) GCWeb/assets/wmms-spl.svg

(I wonder if we are in one of the "departments" where a communication about the changes was sent to. 😉)

The above changes can be found in commit 63c3117 which merged from Pull Request #2024. The diff for templates/splash/layouts/splashpage-en.html and templates/splash/layouts/splashpage-fr.html does show something useful though:

<img src="{{ setting-resourcesBasePathTheme }}/assets/sig-spl.svg" width="283" alt="Government of Canada" property="logo" /><span class="wb-inv"> / <span lang="fr">Gouvernement du Canada</span></span>
<img src="{{ setting-resourcesBasePathTheme }}/assets/sig-blk-en.svg" width="283" alt="Government of Canada" property="logo" /><span class="wb-inv"> / <span lang="fr">Gouvernement du Canada</span></span>
<img src="{{ setting-resourcesBasePathTheme }}/assets/wmms-spl.svg" width="127" alt="Symbol of the Government of Canada" /><span class="wb-inv"> / <span lang="fr">Symbole du gouvernement du Canada</span></span>
<img src="{{ setting-resourcesBasePathTheme }}/assets/wmms-blk.svg" width="127" alt="Symbol of the Government of Canada" /><span class="wb-inv"> / <span lang="fr">Symbole du gouvernement du Canada</span></span>

In summary:

  1. https://wet-boew.github.io/themes-dist/GCWeb/GCWeb/assets/sig-spl.svg becomes

  2. https://wet-boew.github.io/themes-dist/GCWeb/GCWeb/assets/wmms-spl.svg becomes

@duboisp
Copy link
Member

duboisp commented Oct 27, 2022

@DamonU2 and @anthonyfok can you send us an email to [email protected] so I will be able to add your name into the message we are trying to send with more details about that breaking change and the we are on our way to prepare another one with the changes to make into the site footer.

Side note: You can see more details in our recent draft to update for our public versioning API (DD-14)

@duboisp
Copy link
Member

duboisp commented Nov 10, 2022

@DamonU2 we re-added the splash page images into our distribution zip file. The previous releases have back that assets included.

We are now going to include in our versioning API files assets required after the implementation of GCWeb (sites, templates, components), like the splash page images.

We are going to mark those assets as deprecated and it will remains in our distribution for at least one year to allow department to migrate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants