-
Notifications
You must be signed in to change notification settings - Fork 1
[PER-10275] Fixed archive selector modal available archives list #718
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
base: main
Are you sure you want to change the base?
[PER-10275] Fixed archive selector modal available archives list #718
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #718 +/- ##
==========================================
+ Coverage 45.12% 45.34% +0.21%
==========================================
Files 370 371 +1
Lines 11314 11317 +3
Branches 1867 1866 -1
==========================================
+ Hits 5106 5132 +26
+ Misses 6033 6009 -24
- Partials 175 176 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good - I see the other issues you mentioned that are outside the scope of this ticket, mainly the other modal opening behind this one.
It also looks to me like you moved this from core
to shared
, which gives me a little pause because generally the goal had been to move things out of shared, since that module is so big. Can you tell me a little about your thinking there?
src/app/shared/components/archive-switcher/archive-switcher.component.ts
Show resolved
Hide resolved
src/app/core/components/archive-switcher/archive-switcher.component.spec.ts
Show resolved
Hide resolved
I wasn't aware we're trying to move away from 'shared'. I moved it here because part of the bug was that the archive switcher was importing most of its components from shared and they were not rendering(tangled module imports). I can try and move it to a more relevant module, but from my point of view, that wouldn't be core, maybe 'shares'? I could also try to just move this specific component and in time to try and move all relevant component to 'shares'. |
029d7dc
to
5cece18
Compare
@omnignorant @cecilia-donnelly I have added the steps to follow in order to test this bug and also a video in the description of this PR, but if you need more information, please let me know. |
5cece18
to
f087c11
Compare
f087c11
to
ffa944a
Compare
BUG: https://permanent.atlassian.net/browse/PER-10275
OBSERVATION: This bug will only fix the list of archives, but there are other bugs that will affect this flow. Will add those as new issues and prioritize them in the backlog.
STEPS TO TEST:
EXPECTED RESULT:
A modal should appear, with a list of the archives from account B.
OBSERVATION:
The currently selected archive for account B should not be in the list.
I have created this video that replicates the steps to test(can only view the video in Chrome Browser):
show-archives-in-archive-selector.webm