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

feat: Add a toggle between hex and base64 #62

Merged
merged 4 commits into from
Apr 11, 2023
Merged

feat: Add a toggle between hex and base64 #62

merged 4 commits into from
Apr 11, 2023

Conversation

gvking
Copy link
Contributor

@gvking gvking commented Apr 3, 2023

A lot less rewrite -> same feature of toggle. Also makes use of session storage instead of chrome storage so we don't request extra permissions from the users, and is more lightweight and easier to access.

@gvking gvking requested review from joeyparrish and beback4u April 3, 2023 19:48
Copy link
Contributor

@beback4u beback4u left a comment

Choose a reason for hiding this comment

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

You will follow the release process: go/eme-logger-formatter-release.

  1. Do integration tests before landing this change. See Test by developer section.
  2. Update after landing this change. Find the test tracker link from Test trackers section. Add new test steps for hex<->base64 toggle for "1.1.1 - EME Logger Function Tests".
  3. Follow the publishing process for EME Logger. Note that the release-please branch by automated release tool will pick your change up once landed, I think.

@gvking gvking removed the request for review from joeyparrish April 7, 2023 18:19
Copy link
Contributor

@beback4u beback4u left a comment

Choose a reason for hiding this comment

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

LGTM

@gvking gvking merged commit 7fa1a98 into shaka-project:main Apr 11, 2023
gvking pushed a commit that referenced this pull request Apr 11, 2023
🤖 I have created a release *beep* *boop*
---


##
[3.4.0](v3.3.0...v3.4.0)
(2023-04-11)


### Features

* Add a toggle between hex and base64
([#62](#62))
([7fa1a98](7fa1a98))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added the status: archived Archived and locked; will not be updated label Jul 25, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: archived Archived and locked; will not be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants