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

Develop #869

Merged
merged 50 commits into from
Nov 12, 2022
Merged

Develop #869

merged 50 commits into from
Nov 12, 2022

Conversation

jaedb
Copy link
Owner

@jaedb jaedb commented Sep 25, 2022 •
edited
Loading

  • Upgrading all dependencies, notably:
    • React Router
    • Jest
    • Moving from npm to yarn
  • Removing defunct sorting, all using ReactDND now
  • ...

DavisNT and others added 26 commits July 18, 2022 20:55
@DavisNT
Add Tidal source for Tracks
2bc3059
@jaedb
Fixing hooks order
5715c2c
@jaedb
Merge branch 'master' into develop
80b0340
@jaedb
Upgrading from ReactSortable to ReactDND
ae3b831
@jaedb
Removing sources priority sorting (deprecating it)
daee7b4
@jaedb
Reinstating page navigation hotkeys, fixes #862
06727bf
@jaedb
Merging PR #853
33bb054
@jaedb
Merging develop
fcacafd
@jaedb
Correctly excluding buildout files (TESTING REQ'D FOR ACTIONS)
310fe04
@jaedb
Upgrading to dispatch
b3dc8f8
@jaedb
Removing item from store when loading fails
8107372
@jaedb
Allowing dropdown to break out of overflow, fixes #827
3c55761
@jaedb
Simplifying click-outside-of behavior, fixes #827
d165aa6
@jaedb
Browse URI + name syntax, fixes #829
0f950ed
@jaedb
Removing iris:browse: URI prefix, fixes #829
e3989c2
@jaedb
Wrapping CSS calcs in calc(); TODO: Remove FontAwesome
9633a89
@jaedb
NPM easy upgrade, fixes #865
2d77032
@jaedb
Merge branch 'maint/853-merging-and-cleaning' into develop
d97636b
@jaedb
Adding missing params wrapper
4526b64
@nawordar @cezdro
Fix selection ordering
86a6ade
@cezdro
Merge branch 'jaedb:master' into fix-selection
5be8822
@jaedb
Merge pull request #864 from nawordar/fix-selection
a4b9216 
Fix selection ordering
@jaedb
Fixing Router upgrade issues
3e27921
@jaedb
Merge branch 'develop' of github.com:jaedb/Iris into develop
cd94977
@jaedb
Merge pull request #854 from DavisNT/tt
5562ad9 
Add Tidal source for Tracks
@jaedb
Merge branch 'develop' of github.com:jaedb/Iris into develop
129ed32
@codecov
Copy link

codecov bot commented Sep 25, 2022 •
edited
Loading

Codecov Report

Merging #869 (5529ed7) into master (0182112) will decrease coverage by 16.69%.
The diff coverage is 12.04%.

@@             Coverage Diff             @@
##           master     #869       +/-   ##
===========================================
- Coverage   28.38%   11.68%   -16.70%     
===========================================
  Files           6      168      +162     
  Lines         842     9710     +8868     
  Branches        0       73       +73     
===========================================
+ Hits          239     1135      +896     
- Misses        603     8575     +7972
Impacted Files Coverage Δ
src/js/components/AddedFrom.js 0.00% <0.00%> (ø)
src/js/components/ContextMenu/ContextMenu.js 16.00% <ø> (ø)
src/js/components/ContextMenu/ContextMenuItems.js 12.50% <0.00%> (ø)
src/js/components/ErrorMessage.js 0.00% <ø> (ø)
src/js/components/Fields/Dropzones.js 12.00% <0.00%> (ø)
src/js/components/Fields/OutputControl.js 8.06% <0.00%> (ø)
src/js/components/Fields/SearchForm.js 3.03% <0.00%> (ø)
src/js/components/Hotkeys.js 0.87% <0.00%> (ø)
src/js/components/Icon.js 62.50% <ø> (ø)
src/js/components/LinksSentence.js 81.25% <ø> (ø)
... and 220 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@jaedb
Snapshot of actual Album; App test unweidly, TODO: achieve coverage f…
bcc0550 
…urther down DOM
@jaedb
Reinstating package-lock.json for npm-dependent workflows
857a3bf
@jaedb
Upgrading Debug to functional component
0e36684
@jaedb
Upgrading webpack dev server
84bc77a
@jaedb
Dynamic basename ('/' or '/iris/') depending on initial path; Webpack…
7564c1b 
… dev server fully functional
@jaedb
Removing tracks from Album (for now); Jest and react-dnd not getting …
84db4e0 
…along
@jaedb
Limiting workers for jest
c034788
@jaedb
Upgrading to node 14
a926882
@jaedb
Snapshot excludes tracks (due to complex Jest + DND limitations)
275143d
@jaedb
Moving loader to thumbnail so it doesn't block underlying render, mit…
742a0e9 
…igates impact of #837
@jaedb
Thumbnail doubles up as secondary loader; Webpack dev server as prima…
bb456a9 
…ry dev stack, dynamic base_href to accommodate
@jaedb
Removing blanket 'loading: true' when LOAD_ITEM called
fc5376b
@jaedb
Track loading thumbnail; 'Go to track' context item
43734f8
@jaedb
Only build HTML for WDS, use our custom builder for actual builds
e906177
@jaedb
Remove absolute ref for service worker
627ece5
@jaedb
Removing more absolute references
f0b10f4
@jaedb
Removing more absolute asset references
02aee0b
@jaedb
Volumes restructure to sample docker-compose.yml
074b97c 
- Playlist persistence
- Consistent structure to match that of /var/lib/mopidy/
@jaedb
Moving HTML builder into webpack
154e722 
- Allows handling of template variables
- Retaining test_mode switch for hot-swap JS bundle
@jaedb
Restructuring docker (again)
60c4a4e 
- Nested structure to match /var/lib/mopidy
- Updating gitignore to exclude individual configs
@jaedb
Tox command
5529ed7 
- Ditching 'sudo', no longer needed with newer docker setup
- Excluding /docker folder from package
@jaedb jaedb marked this pull request as ready for review November 12, 2022 06:57
@jaedb jaedb merged commit a4f5d89 into master Nov 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jaedb @DavisNT @nawordar @cezdro