Skip to content

Improve package manager UX#1710

Open
samamou wants to merge 4 commits intomasterfrom
1701-improve-package-manager-ux
Open

Improve package manager UX#1710
samamou wants to merge 4 commits intomasterfrom
1701-improve-package-manager-ux

Conversation

@samamou
Copy link
Member

@samamou samamou commented Feb 6, 2025

closes issue #1701

@samamou samamou linked an issue Feb 6, 2025 that may be closed by this pull request
Improve package manager UX #1701
Open
4 tasks
@samamou samamou changed the title Add hardcoded categories Improve package manager UX Feb 6, 2025
@samamou samamou mentioned this pull request Feb 6, 2025
Open
7 tasks
samamou
samamou commented Feb 7, 2025
View reviewed changes
src/plugins/score-plugin-packagemanager/PackageManager/View.cpp Outdated
return;
QNetworkRequest rqst{
QUrl("https://raw.githubusercontent.com/ossia/score-addons/master/addons.json")};
QUrl("https://raw.githubusercontent.com/ossia/score-addons/refs/heads/"
Copy link
Member Author

@samamou samamou Feb 7, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • TODO: Change back when this branch is merged with main

Copy link
Member Author

@samamou samamou Apr 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is still pointing to my branch for testing purposes, should point to master when merged

Copy link
Member

@jcelerier jcelerier Apr 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when you are ready to merge add a last commit that does this change!

samamou
samamou commented Feb 11, 2025
View reviewed changes
src/plugins/score-plugin-packagemanager/PackageManager/Presenter.cpp Outdated

ps_view.remoteView()->setSelectionModel(&ps_model.remoteSelection);

// ps_view.remoteView()->setSelectionModel(&ps_model.remoteSelection);
Copy link
Member Author

@samamou samamou Feb 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to re-enable this for the install button to work

@samamou samamou marked this pull request as ready for review February 13, 2025 16:08
@samamou
Copy link
Member Author

samamou commented Feb 27, 2025 •
edited
Loading

Can we please check why the checks are failing so we can merge soon? @jcelerier
image

@jcelerier jcelerier force-pushed the 1701-improve-package-manager-ux branch from abedc99 to c2c00cf Compare March 22, 2025 14:42
@jcelerier
Copy link
Member

jcelerier commented Mar 22, 2025

I just rebased on current master which has fully green CI so that we can see if there's any failure related to this patch specifically :)

@jcelerier
Copy link
Member

jcelerier commented Mar 23, 2025

CI is green in all the places that matter :)

@jcelerier
Copy link
Member

jcelerier commented Apr 1, 2025

what else do we need here?

@jcelerier
Copy link
Member

jcelerier commented Apr 3, 2025

aaaaa never git merge :`)

$ git checkout master
$ git pull
$ git checkout -
$ git rebase master
<it will likely skip the last commit since it's redundant now>
$ git push --force

@samamou samamou force-pushed the 1701-improve-package-manager-ux branch from 687c9a0 to 8ce4374 Compare April 3, 2025 06:40
jcelerier
jcelerier approved these changes Apr 3, 2025
View reviewed changes
Copy link
Member

@jcelerier jcelerier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

code looks good! what is remaining?

src/plugins/score-plugin-packagemanager/PackageManager/View.cpp
m_progress->setValue(((bytesReceived / 1024.) / (bytesTotal / 1024.)) * 100);
}

void PluginSettingsView::updateCategoryComboBox(int tabIndex)
Copy link
Member

@jcelerier jcelerier Apr 3, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 looking good!

@samamou
Copy link
Member Author

samamou commented Apr 4, 2025

nothing! can we merge? :) @jcelerier

@jcelerier
Copy link
Member

jcelerier commented Apr 4, 2025

gonna test-drive it locally and then yes ! :)

@jcelerier
Copy link
Member

jcelerier commented Apr 4, 2025

a
hmm when I set the combobox, I don't see any change, what is supposed to happen ?

@samamou
Copy link
Member Author

samamou commented Apr 5, 2025

Ypu don't see any change because it wasn't implemented 🙈 kept it on pause.....

After the last commit, this should be the behavior. can you please test one last time? @jcelerier

Screen.Recording.2025-04-05.at.6.48.57.PM.mov

@samamou samamou requested a review from jcelerier April 5, 2025 22:55
jcelerier
jcelerier approved these changes May 5, 2025
View reviewed changes
Copy link
Member

@jcelerier jcelerier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woooops I had done the review but forgot to submit it

src/plugins/score-plugin-packagemanager/PackageManager/View.cpp Outdated
return;
QNetworkRequest rqst{
QUrl("https://raw.githubusercontent.com/ossia/score-addons/master/addons.json")};
QUrl("https://raw.githubusercontent.com/ossia/score-addons/refs/heads/"
Copy link
Member

@jcelerier jcelerier Apr 6, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

when you are ready to merge add a last commit that does this change!

@jcelerier jcelerier force-pushed the 1701-improve-package-manager-ux branch from b1f73c0 to 83b615b Compare December 30, 2025 04:54
@jcelerier
Copy link
Member

jcelerier commented Dec 30, 2025

image

@jcelerier jcelerier force-pushed the master branch 3 times, most recently from 86a1ef8 to a402674 Compare December 31, 2025 15:10
@jcelerier jcelerier force-pushed the master branch 6 times, most recently from ff76fa0 to 1e41cd2 Compare January 9, 2026 22:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jcelerier jcelerier jcelerier 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.

Improve package manager UX

2 participants

@samamou @jcelerier