Skip to content

LG-4449 Implement rules of use for existing users#5040

Merged
stevegsa merged 32 commits intomainfrom
stevegsa-rules-of-use-for-existing-users
May 22, 2021
Merged

LG-4449 Implement rules of use for existing users#5040
stevegsa merged 32 commits intomainfrom
stevegsa-rules-of-use-for-existing-users

Conversation

@stevegsa
Copy link
Contributor

@stevegsa stevegsa commented May 7, 2021

Screen Shot 2021-05-06 at 10 14 55 PM

@stevegsa stevegsa force-pushed the stevegsa-rules-of-use-for-existing-users branch from f8d3539 to cc4934c Compare May 7, 2021 03:40
Copy link
Contributor

@aduth aduth left a comment

Choose a reason for hiding this comment

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

A suggestion at #5040 (comment) that I think is worth including, but otherwise LGTM 👍

Copy link
Contributor

@mitchellhenke mitchellhenke left a comment

Choose a reason for hiding this comment

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

👏🏼 👏🏼 👏🏼

@aduth
Copy link
Contributor

aduth commented May 18, 2021

FYI #5066 changed how locale YAML files are normalized, so you might want to rebase / merge to check for any local changes.

# git fetch origin
# git rebase origin/main
yarn install
make normalize_yaml

@stevegsa stevegsa force-pushed the stevegsa-rules-of-use-for-existing-users branch from 78719bf to 0f02fec Compare May 21, 2021 18:50
@stevegsa stevegsa force-pushed the stevegsa-rules-of-use-for-existing-users branch from a6a27b2 to 3b482f1 Compare May 22, 2021 05:23
@stevegsa stevegsa merged commit 534dc5a into main May 22, 2021
@stevegsa stevegsa deleted the stevegsa-rules-of-use-for-existing-users branch May 22, 2021 05:44
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