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.

Sign up for GitHub

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

Jump to bottom

re-queries the API when language is changed #940

Merged
merged 5 commits into from
May 31, 2024
Merged

re-queries the API when language is changed #940

merged 5 commits into from
May 31, 2024

Conversation

Razzmatazzz
Copy link
Member

Resolves #939

@Razzmatazzz Razzmatazzz requested a review from a team as a code owner May 22, 2024 15:59
@github-actions GitHub Actions
Copy link
Contributor

👋 Thanks for opening a pull request!

If you are new, please check out the trimmed down summary of our deployment process below:

  1. 👀 Observe the CI jobs and tests to ensure they are passing

  2. ✔️ Obtain an approval/review on this pull request

  3. 🚀 Deploy your pull request to the development environment with .deploy to development

  4. 🚀 Deploy your pull request to the production environment with .deploy

    If anything goes wrong, rollback with .deploy main

  5. 🎉 Merge!

Note: If you have a larger change and want to block deployments, you can run .lock --reason <reason> to lock all other deployments (remove with .unlock)

You can view the branch deploy usage guide for additional information

@Razzmatazzz
Copy link
Member Author

.deploy

@github-actions GitHub Actions
Copy link
Contributor

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to production

You can watch the progress here 🔗

Branch: query-on-language-change

@github-actions GitHub Actions
Copy link
Contributor

Deployment Results ✅

Razzmatazzz successfully deployed branch query-on-language-change to production

Environment URL: tarkov.dev

@Shebuka
Copy link
Contributor

Shebuka commented May 23, 2024

.deploy main

@github-actions GitHub Actions
Copy link
Contributor

⚠️ Cannot claim deployment lock

Sorry Shebuka, the production environment deployment lock is currently claimed by Razzmatazzz

Lock Details 🔒

  • Environment: production
  • Branch: query-on-language-change
  • Created At: 2024-05-23T00:45:28.113Z
  • Created By: Razzmatazzz
  • Sticky: true
  • Global: false
  • Comment Link: click here
  • Lock Link: click here

The current lock has been active for 0d:6h:14m:29s

If you need to release the lock, please comment .unlock production

@Shebuka
Copy link
Contributor

Shebuka commented May 23, 2024

Something not right with this PR, site now hangs for a couple of seconds when API returns data

@Shebuka
Copy link
Contributor

Shebuka commented May 23, 2024

.unlock production

@Shebuka
Copy link
Contributor

Shebuka commented May 23, 2024

.deploy main

@github-actions GitHub Actions
Copy link
Contributor

🔓 Deployment Lock Removed

The production deployment lock has been successfully removed

@github-actions GitHub Actions
Copy link
Contributor

Deployment Triggered 🚀

Shebuka, started a branch deployment to production

You can watch the progress here 🔗

Branch: main

@github-actions GitHub Actions
Copy link
Contributor

Deployment Results ✅

Shebuka successfully deployed branch main to production

Environment URL: tarkov.dev

@Razzmatazzz
Copy link
Member Author

.deploy to development

@github-actions GitHub Actions
Copy link
Contributor

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to development

You can watch the progress here 🔗

Branch: query-on-language-change

@github-actions GitHub Actions
Copy link
Contributor

Deployment Results ✅

Razzmatazzz successfully deployed branch query-on-language-change to development

Show Results

https://f906b5e0.tarkov-dev.pages.dev

@Razzmatazzz
Copy link
Member Author

@Shebuka Does the hanging issue seem fixed now? https://preview.tarkov-dev.pages.dev/

@Shebuka
Copy link
Contributor

Shebuka commented May 23, 2024

@Razzmatazzz seems feexed 👍

@Razzmatazzz
Copy link
Member Author

.deploy

@github-actions GitHub Actions
Copy link
Contributor

⚠️ Cannot claim deployment lock

Sorry Razzmatazzz, the production environment deployment lock is currently claimed by Shebuka

Lock Details 🔒

  • Environment: production
  • Branch: main
  • Created At: 2024-05-23T07:01:30.020Z
  • Created By: Shebuka
  • Sticky: true
  • Global: false
  • Comment Link: click here
  • Lock Link: click here

The current lock has been active for 0d:9h:1m:35s

If you need to release the lock, please comment .unlock production

@Razzmatazzz
Copy link
Member Author

.unlock production

@github-actions GitHub Actions
Copy link
Contributor

🔓 Deployment Lock Removed

The production deployment lock has been successfully removed

@Razzmatazzz
Copy link
Member Author

.deploy

@github-actions GitHub Actions
Copy link
Contributor

Deployment Triggered 🚀

Razzmatazzz, started a branch deployment to production

You can watch the progress here 🔗

Branch: query-on-language-change

@github-actions GitHub Actions
Copy link
Contributor

Deployment Results ✅

Razzmatazzz successfully deployed branch query-on-language-change to production

Environment URL: tarkov.dev

@Razzmatazzz Razzmatazzz merged commit 0887623 into main May 31, 2024
2 checks passed
@Razzmatazzz Razzmatazzz deleted the query-on-language-change branch May 31, 2024 20:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Shebuka Shebuka Shebuka approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update API data on language change
2 participants
@Razzmatazzz @Shebuka