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

[PLATFORM-3432] Update fork with upstream (v1.1.3) #7

Merged
merged 23 commits into from
Jul 17, 2024

Conversation

Evanjt1
Copy link

@Evanjt1 Evanjt1 commented Jul 17, 2024

PLATFORM-3432

Latest version of upstream (v1.1.3) has some updates to the login sync api that we'll need for some upcoming features. This PR brings our fork up to current head of upstream.

@Evanjt1 Evanjt1 added the enhancement New feature or request label Jul 17, 2024
@Evanjt1 Evanjt1 self-assigned this Jul 17, 2024
Copy link

@aflesher aflesher left a comment

Choose a reason for hiding this comment

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

LGTM

"build": "rimraf dist && tsc -p tsconfig.prod.json",
"release": "npm run build && ./scripts/release.sh",
"playwright": "npm run build && docker-compose -f docker-compose.playwright.yml up --build -d && cd playwright && playwright test && docker-compose -f ../docker-compose.playwright.yml down",
"playwright": "npm run build && docker compose -f docker-compose.playwright.yml up --build -d && cd playwright && playwright test && docker compose -f ../docker-compose.playwright.yml down",

Choose a reason for hiding this comment

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

Is this a recent change to docker? I've always used docker-compose

Copy link
Author

Choose a reason for hiding this comment

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

as --> sdk: {error=access_denied}
deactivate as
deactivate sdk
@enduml

Choose a reason for hiding this comment

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

Great to see this plantuml diagrams added to the source. Not a blocker but we should add an image of the rendered diagrams to the source like our other repos.

@Evanjt1 Evanjt1 merged commit 44301f5 into main Jul 17, 2024
@Evanjt1 Evanjt1 deleted the feat/update-fork/PLATFORM-3432 branch July 17, 2024 22:25
@Evanjt1 Evanjt1 restored the feat/update-fork/PLATFORM-3432 branch July 17, 2024 22:28
Evanjt1 added a commit that referenced this pull request Jul 17, 2024
Evanjt1 added a commit that referenced this pull request Jul 17, 2024
@Evanjt1 Evanjt1 deleted the feat/update-fork/PLATFORM-3432 branch July 17, 2024 22:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants