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

Release v1.100.0 - develop <- master #9574

Merged
merged 13 commits into from
Aug 21, 2023
Merged

Release v1.100.0 - develop <- master #9574

merged 13 commits into from
Aug 21, 2023

Conversation

mjac0bs
Copy link
Contributor

@mjac0bs mjac0bs commented Aug 21, 2023

Following the v1.100.0 release.

hana-akamai and others added 13 commits August 11, 2023 11:21
## Description 📝
Follow up to #9494. The previous PR fixed the Cloud crashing issue, but introduced another issue where we were not unescaping the encoded user data. So, when running the `atob` function on the user data, the output string included `%`s instead of just the original string.

reading:
https://base64.guru/developers/javascript/examples/unicode-strings
https://stackoverflow.com/questions/30631927/converting-to-base64-in-javascript-without-deprecated-escape-call (see the second answer by `T S`)

## How to test 🧪
- Add a console log after line 706 to log the user data `console.log(utoa(this.props.userData))`
- Follow the steps in #9494 
- Take the result of the console log and run `atob`
- The result should be readable and not have unexpected characters such as `%`
Release v1.99.1 - staging → master
Release v1.100.0 - release → staging
Release v1.100.0 - staging → master
@mjac0bs mjac0bs self-assigned this Aug 21, 2023
@bnussman-akamai bnussman-akamai added Add'tl Approval Needed Waiting on another approval! and removed Ready for Review labels Aug 21, 2023
@cypress
Copy link

cypress bot commented Aug 21, 2023

2 flaky tests on run #5046 ↗︎

0 196 5 0 Flakiness 2

Details:

Merge pull request #9571 from linode/staging
Project: Cloud Manager E2E Commit: 49a2df205f
Status: Passed Duration: 15:14 💡
Started: Aug 21, 2023 7:09 PM Ended: Aug 21, 2023 7:25 PM
Flakiness  account/service-transfer.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Account service transfers > can initiate and cancel a service transfer Output Screenshots Video
Flakiness  linodes/rescue-linode.spec.ts • 1 flaky test

View Output Video

Test Artifacts
Rescue Linodes > Can reboot a Linode into rescue mode Output Screenshots Video

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

Copy link
Member

@bnussman-akamai bnussman-akamai left a comment

Choose a reason for hiding this comment

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

Nice!! No conflicts?

Don't forget to merge commit this one!

@bnussman-akamai bnussman-akamai added Approved Multiple approvals and ready to merge! and removed Add'tl Approval Needed Waiting on another approval! labels Aug 21, 2023
@mjac0bs mjac0bs merged commit 81864b0 into develop Aug 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Approved Multiple approvals and ready to merge! Master → Develop Updating develop with master
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants