Skip to content

Conversation

@cin-bun
Copy link
Collaborator

@cin-bun cin-bun commented May 26, 2025

Checklist:

  • The ticket number is indicated in the comments
  • Unrelated code has been removed (unused imports, print statements, unrelated template context variables, commented code, etc)
  • I translated the new messages and added only the .po files that actually changed.
  • If there were changes in the frontend, there is a PR in the front repository.

  • I have added tests that prove my code works
  • No common performance issues (e.g. N + 1 problem)

@cin-bun cin-bun requested a review from Dmi4er4 May 26, 2025 12:36
@cin-bun cin-bun merged commit 076c84b into cscenter:master May 26, 2025
2 checks passed
cin-bun added a commit to cin-bun/lms that referenced this pull request May 27, 2025
* fix: getting site id from settings not request

* fix: site id

* fix: tests

* fix: arc

* fix: cr

* fix: cr

# Checklist:

- [ ] The **ticket number** is indicated in the comments
- [ ] Unrelated code has been removed **(unused imports, `print` statements, unrelated template context variables, commented code, etc)**
- [ ] I **translated the new messages** and added only the .po files that actually changed.
- [ ] If there were **changes in the frontend**, there is a PR in the front repository.
---
- [ ] I have added **tests** that prove my code works
- [ ] No common performance issues **(e.g. N + 1 problem)**
cin-bun added a commit that referenced this pull request May 27, 2025
* fix: getting site id from settings not request

* fix: site id

* fix: tests

* fix: arc

* fix: cr

* fix: cr

# Checklist:

- [ ] The **ticket number** is indicated in the comments
- [ ] Unrelated code has been removed **(unused imports, `print` statements, unrelated template context variables, commented code, etc)**
- [ ] I **translated the new messages** and added only the .po files that actually changed.
- [ ] If there were **changes in the frontend**, there is a PR in the front repository.
---
- [ ] I have added **tests** that prove my code works
- [ ] No common performance issues **(e.g. N + 1 problem)**
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants