Skip to content

Add more data to get_package_data(), renamed from get_update_data()#286

Merged
afragen merged 10 commits intofairpm:mainfrom
afragen:get-package-data
Oct 9, 2025
Merged

Add more data to get_package_data(), renamed from get_update_data()#286
afragen merged 10 commits intofairpm:mainfrom
afragen:get-package-data

Conversation

@afragen
Copy link
Copy Markdown
Contributor

@afragen afragen commented Oct 4, 2025

Rename get_update_data() to get_package_data().

Add data elements needed for plugins_api or themes_api filter.

Signed-off-by: Andy Fragen <andy@thefragens.com>
Signed-off-by: Andy Fragen <andy@thefragens.com>
@afragen afragen requested review from Ipstenu, costdev and rmccue October 4, 2025 21:08
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 4, 2025

Signed-off-by: Andy Fragen <andy@thefragens.com>
@costdev
Copy link
Copy Markdown
Member

costdev commented Oct 4, 2025

Looks like there's a breakage in the PR:

  1. Go to Plugins > Add New.
  2. Install a plugin by DID, for example, did:plc:e3rm6t7cspgpzaf47kn3nnsl.
  3. Unbump its version.
  4. Go back to Plugins > Add New and search for the DID again.
  5. 🐛 The button says "Install Now" rather than "Update Now".

@afragen
Copy link
Copy Markdown
Contributor Author

afragen commented Oct 4, 2025

Can confirm but this is only in the plugin card. Need to look into plugin card and button creation.

Signed-off-by: Andy Fragen <andy@thefragens.com>
@costdev
Copy link
Copy Markdown
Member

costdev commented Oct 5, 2025

It's a bit weird, because when I did this, the "Update now" button appeared as expected. Not saying that's the right approach, but that data passes on to the determination of what the status is, or which button to show.

Signed-off-by: Andy Fragen <andy@thefragens.com>
Signed-off-by: Andy Fragen <andy@thefragens.com>
Signed-off-by: Andy Fragen <andy@thefragens.com>
@afragen afragen marked this pull request as ready for review October 5, 2025 22:10
Signed-off-by: Andy Fragen <andy@thefragens.com>
Signed-off-by: Andy Fragen <andy@thefragens.com> 

Signed-off-by: Andy Fragen <andy@thefragens.com>
Signed-off-by: Andy Fragen <andy@thefragens.com>
@afragen afragen merged commit bef2c51 into fairpm:main Oct 9, 2025
46 checks passed
@afragen afragen deleted the get-package-data branch October 9, 2025 20:00
afragen added a commit to costdev/fair-plugin that referenced this pull request Oct 15, 2025
Signed-off-by: Andy Fragen <andy@thefragens.com>

Signed-off-by: Andy Fragen <andy@thefragens.com>
@Ipstenu Ipstenu added this to the 1.1 milestone Oct 27, 2025
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.

3 participants