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.

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

Add Jobs for Product Creation, Builds, and Publishing #1060

Open
wants to merge 44 commits into
base: feature/svelte
Choose a base branch
from

Conversation

FyreByrd
Copy link

@FyreByrd FyreByrd commented Dec 6, 2024

Handles the creation of a Product in BuildEngine, as well as the Build and Release processes in BuildEngine.
Properly handles the creation of ProductArtifacts in the database.
Also added UI for creating a Product.

image

image

image

image

@FyreByrd
Copy link
Author

FyreByrd commented Dec 6, 2024

A few things I am not certain on.

  1. Do the modal boxes for product creation need a visible cancel button?
  2. How should StoreLanguages be handled?

I am also experiencing some weird bugs with BuildEngine right now.

For these reasons, this is currently a draft PR.

@FyreByrd
Copy link
Author

FyreByrd commented Dec 6, 2024

One other thing that should be resolved first: What does the expired status mean and how should it be handled?

- Tighten typing of environment
- Create bullmq job to get versionCode for GOOGLE_PLAY_UPLOADED, because XState won't properly handle asynchronous actions (ugh)
why was this async in the first place???
yes, it was my fault, but still, why???
The activityName was sometimes showing up blank. After investigating further, I was able to narrow it down to existing just at the start of a new product. This issue was sometimes persisting when the jobs backend was incorrect.
Had been deleting previous artifacts based on logic from S1 backend. This was determined to be unneeded, as the relevant code in S1 was determined to be unreachable after talking with @chrisvire
@FyreByrd FyreByrd force-pushed the feature/product-jobs branch from 27eb419 to 44c04b5 Compare December 6, 2024 20:54
@FyreByrd FyreByrd marked this pull request as ready for review December 6, 2024 21:05
@FyreByrd
Copy link
Author

FyreByrd commented Dec 6, 2024

I resolved the BuildEngine issue, so this should be ready for review/requests

@FyreByrd FyreByrd requested a review from chrisvire December 6, 2024 21:05
@chrisvire
Copy link
Member

  1. Do the modal boxes for product creation need a visible cancel button?

S1 has a close X in the top right.

@FyreByrd
Copy link
Author

  1. Do the modal boxes for product creation need a visible cancel button?

S1 has a close X in the top right.

Added

Switch updateProjectDateActive back to expected functionality based on S1.

I still don't fully understand under which conditions we would want to be running this. Right now I just have it to where it will execute each time a product is created, updated, or deleted.
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