Skip to content

Remove scratch projects scope filter #542

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

Closed
wants to merge 6 commits into from
Closed

Conversation

loiswells97
Copy link
Contributor

Status

  • Closes add issue numbers or delete
  • Related to add issue numbers or delete

Points for consideration:

  • Security
  • Performance

What's changed?

Description of what's been done - bullets are often best

Steps to perform after deploying to production

If the production environment requires any extra work after this PR has been deployed detail it here. This could be running a Rake task, a migration, or upgrading a Gem. That kind of thing.

@cla-bot cla-bot bot added the cla-signed label May 7, 2025
@loiswells97 loiswells97 temporarily deployed to editor-api-p-show-scrat-slzb7h May 7, 2025 13:43 Inactive
@loiswells97 loiswells97 temporarily deployed to editor-api-p-show-scrat-slzb7h May 7, 2025 13:48 Inactive
@loiswells97 loiswells97 self-assigned this May 7, 2025
@adrian-rpf
Copy link

HOLD for the moment until we are ready to show Scratch

@Drayden237
Copy link

@loiswells97 this is not blocked now right? Do you mind moving it back to in progress and finalising this work during this sprint?

@loiswells97
Copy link
Contributor Author

@loiswells97 this is not blocked now right? Do you mind moving it back to in progress and finalising this work during this sprint?

I think GFR are now doing this? And we're doing the frontend safeguards to smooth this over? Unless I've misunderstood things 🤔

@adrian-rpf
Copy link

@loiswells97 this is not blocked now right? Do you mind moving it back to in progress and finalising this work during this sprint?

I think GFR are now doing this? And we're doing the frontend safeguards to smooth this over? Unless I've misunderstood things 🤔

Correct. See #558

@Drayden237
Copy link

Should this ticket be removed then? @loiswells97 @adrian-rpf

@adrian-rpf
Copy link

CLOSE as no longer needed

@adrian-rpf adrian-rpf closed this Jun 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants