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

task/WD-6598 #13211

Closed
wants to merge 7 commits into from
Closed

task/WD-6598 #13211

wants to merge 7 commits into from

Conversation

lizzochek
Copy link
Contributor

@lizzochek lizzochek commented Oct 5, 2023

Done

  • Updated link and icon for 23.10

QA

  • Check out this feature branch
  • Run the site using the command ./run serve or dotrun
  • View the site locally in your web browser at: http://0.0.0.0:8001/
    • Be sure to test on mobile, tablet and desktop screen sizes

Issue / Card

WD-6598

Fixes #

Screenshots

Help

QA steps - Commit guidelines

Akbar Abdrakhmanov and others added 6 commits October 2, 2023 16:27
* Fix subscription status chips

* Remove juju install section

* Address comments

* Refresh /download/raspberry-pi

* Hide images on smaller screens

* Update with UX suggestions

* Remove whitespace from th tag

* Update raspberry-pi 5 image

---------

Co-authored-by: minkyngkm <[email protected]>
Co-authored-by: Carlos Wu Fei <[email protected]>
Co-authored-by: Carlos Wu <[email protected]>
Co-authored-by: Min Kim <[email protected]>
@webteam-app
Copy link

Demo starting at https://ubuntu-com-13211.demos.haus

releases.yaml Outdated
Comment on lines 4 to 11
short_version: "23.10"
full_version: "23.10"
core_version: "22"
release_date: "April 2023"
eol: "January 2024"
eol: "July 2024"
past_eol_date: false
release_notes_url: "https://discourse.ubuntu.com/t/lunar-lobster-release-notes/31910"
release_notes_url: "https://discourse.ubuntu.com/t/mantic-minotaur-release-notes/35534"
lts:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't need any of these changes, we still haven't released 23.10, this will change on October 12

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The copy doc had those changes

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Liza, you need to point your PR to 23-10-release branch instead of main. Then you will see that these changes are already applied there.

@@ -65,7 +65,7 @@ <h2>Ubuntu {{ releases.latest.full_version }}</h2>
<div class="u-align--center u-sv4">
{{
image (
url="https://assets.ubuntu.com/v1/3a32d136-Lobster_circular.png",
url="https://assets.ubuntu.com/v1/d708a347-mantic-minotaur-logo.png",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You may need design review for this.

@carkod carkod requested a review from anthonydillon October 5, 2023 15:26
@lizzochek lizzochek changed the base branch from main to 23-10-release October 5, 2023 15:30
@codecov
Copy link

codecov bot commented Oct 5, 2023

Codecov Report

Merging #13211 (14e5204) into 23-10-release (6dff7df) will increase coverage by 0.09%.
The diff coverage is n/a.

@@                Coverage Diff                @@
##           23-10-release   #13211      +/-   ##
=================================================
+ Coverage          75.30%   75.40%   +0.09%     
=================================================
  Files                106      106              
  Lines               2847     2862      +15     
  Branches             932      942      +10     
=================================================
+ Hits                2144     2158      +14     
- Misses               681      682       +1     
  Partials              22       22              

see 1 file with indirect coverage changes

@@ -101,7 +101,7 @@
"flickity": "^3.0.0",
"formik": "2.2.9",
"intl-tel-input": "17.0.18",
"postcss": "8.4.5",
"postcss": "8.4.31",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This change shouldn't be here

@lizzochek lizzochek changed the base branch from 23-10-release to main October 5, 2023 15:55
@@ -79,7 +79,7 @@ <h2>Ubuntu {{ releases.latest.full_version }}</h2>
<script>performance.mark("Download (Desktop) button rendered")</script>
</p>
<p>
<a href="https://cdimage.ubuntu.com/releases/lunar/release/ubuntu-23.04-desktop-legacy-amd64.iso" onclick="dataLayer.push({'event' : 'GAEvent', 'eventCategory' : 'Download', 'eventAction' : 'Desktop', 'eventLabel' : '{{ releases.latest.short_version }} legacy installer', 'eventValue' : undefined });">Download {{ releases.latest.short_version }} (Legacy Desktop Installer)</a>
<a href="https://cdimage.ubuntu.com/releases/lunar/release/ubuntu-{{ releases.latest.short_version }}-desktop-legacy-amd64.iso" onclick="dataLayer.push({'event' : 'GAEvent', 'eventCategory' : 'Download', 'eventAction' : 'Desktop', 'eventLabel' : '{{ releases.latest.short_version }} legacy installer', 'eventValue' : undefined });">Download {{ releases.latest.short_version }} (Legacy Desktop Installer)</a>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only change you need. To confirm with @anthonydillon

releases.yaml Outdated
Comment on lines 4 to 11
short_version: "23.10"
full_version: "23.10"
core_version: "22"
release_date: "April 2023"
eol: "January 2024"
eol: "July 2024"
past_eol_date: false
release_notes_url: "https://discourse.ubuntu.com/t/lunar-lobster-release-notes/31910"
release_notes_url: "https://discourse.ubuntu.com/t/mantic-minotaur-release-notes/35534"
lts:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Liza, you need to point your PR to 23-10-release branch instead of main. Then you will see that these changes are already applied there.

@lizzochek lizzochek changed the base branch from main to 23-10-release October 6, 2023 07:33
@lizzochek lizzochek requested a review from akbarkz October 6, 2023 08:04
@akbarkz
Copy link
Contributor

akbarkz commented Oct 6, 2023

@lizzochek looks like you didn't rebase your branch agains latest changes. Please rebase it against upstream 23-10-release. As you can see now you have 18 changed files.

@akbarkz
Copy link
Contributor

akbarkz commented Oct 6, 2023

@lizzochek also, it seems like you create a branch in the parent repository, while as you need to create your branch from your forked repository.
Here you can see that both branches are from parent repo:
Screenshot 2023-10-06 at 14 11 36
If you create a branch from your forked repo it will look like this:
Screenshot 2023-10-06 at 14 13 11

@lizzochek lizzochek closed this Oct 6, 2023
@lizzochek lizzochek deleted the task/WD-6598 branch October 6, 2023 08:52
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