Skip to content

app-project: Remove audio + spectrogram slugs from the list of PFE projects - #7498

Merged
goplayoutside3 merged 3 commits into
mainfrom
migrate-audio-spectrogram-projects
Aug 24, 2026
Merged

goplayoutside3 merged 3 commits into
mainfrom
migrate-audio-spectrogram-projects

Conversation

@goplayoutside3

@goplayoutside3 goplayoutside3 commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

Package

app-project

Linked Issue and/or Talk Post

PFE: zooniverse/Panoptes-Front-End#7488
static: zooniverse/static#444

Describe your changes

Remove audio + spectrogram slugs from the list of PFE projects

How to Review

  • Check list against the PFE and static PRs

@goplayoutside3
goplayoutside3 marked this pull request as ready for review August 21, 2026 15:44
@goplayoutside3
goplayoutside3 requested a review from zwolf August 21, 2026 15:54
@@ -84,19 +85,6 @@ const PFE_SLUGS = [
'zooniverse/intro2astro-hubbles-law', // Classrooms
'zooniverse/measuring-the-anzacs', // Classrooms
'zooniverse/zooniverse-in-schools', // Classrooms

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't think trailing commas matter in JS, but there wasn't one before.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Trailing commas don't matter in this file, but this comment is on line 87 'zooniverse/zooniverse-in-schools', // Classrooms. Nothing changed on that line in this PR. Did you mean to reference something else?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah, I see. You're pointing out that there is now a trailing comma at the end of the slugs list before the end of the array ]. It does not matter if there's a trailing comma to the compiler.

@goplayoutside3
goplayoutside3 merged commit 3c33477 into main Aug 24, 2026
6 checks passed
@goplayoutside3
goplayoutside3 deleted the migrate-audio-spectrogram-projects branch August 24, 2026 20:18
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.

2 participants