tech/sendico
2
0
Fork 0
Code Issues 22 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

Organizations now load only once #38

Merged
tech merged 2 commits from SEND002 into main 2025-12-09 18:53:13 +00:00
Conversation 1 Commits 2 Files Changed 1 +3
protuberanets commented 2025-12-08 16:12:40 +00:00
Collaborator
Copy Link

Organizations were being loaded twice because setCurrentOrganization tried to resolve the org using the provider’s existing (empty) list, failed, and left _currentOrg unset, which caused OrganizationLoader to trigger another fetch

Organizations were being loaded twice because setCurrentOrganization tried to resolve the org using the provider’s existing (empty) list, failed, and left _currentOrg unset, which caused OrganizationLoader to trigger another fetch
protuberanets added 1 commit 2025-12-08 16:12:40 +00:00
Organizations now load only once 8115abb569
protuberanets requested review from tech 2025-12-08 16:12:40 +00:00
tech commented 2025-12-08 17:53:02 +00:00
Owner
Copy Link

@protuberanets , это ж не решение. Ты ж подкостылил просто логику. Если установка организации происходит до того, как они загружены, это означает, что где-то логически неверно идет работа с загрузкой организаций. То есть код устроен так, что кто-то пытается ставить организацию до того, как они загружены, а это не нормально. Это не тот случай, когда надо адаптировать поведение провайдера, а когда надо вызывающий код выпрямлять.

@protuberanets , это ж не решение. Ты ж подкостылил просто логику. Если установка организации происходит до того, как они загружены, это означает, что где-то логически неверно идет работа с загрузкой организаций. То есть код устроен так, что кто-то пытается ставить организацию до того, как они загружены, а это не нормально. Это не тот случай, когда надо адаптировать поведение провайдера, а когда надо вызывающий код выпрямлять.
protuberanets added 1 commit 2025-12-09 13:15:45 +00:00
Current org now sets after list gets to the state of the provider 660f689a7a
tech merged commit 1079ad7d0a into main 2025-12-09 18:53:13 +00:00
tech deleted branch SEND002 2025-12-09 18:53:14 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
tech
Labels
Clear labels
backend
bug
Something is not working
 cicd
Continious Integration / Continuous Deployment
 duplicate
This issue or pull request already exists
 enhancement
New feature
 frontend
help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
protuberanets tech
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: tech/sendico#38
Delete Branch "SEND002"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?