Skip to content

Conversation

@yuri-xyz
Copy link
Contributor

@yuri-xyz yuri-xyz commented Mar 21, 2025

Summary of changes

Provide a detailed description of proposed changes.

  • ➕ Implement the /deploy page that allows setting up a blueprint instance's settings for deployment.

Proposed area of change

Put an x in the boxes that apply.

  • apps/tangle-dapp
  • apps/tangle-cloud
  • apps/leaderboard
  • libs/tangle-shared-ui
  • libs/ui-components

Associated issue(s)

Specify any issues that can be closed from these changes (e.g. Closes #233).

Screen Recording

If possible provide screenshots and/or a screen recording of proposed change.

deploy_blueprint_step_1.mov

@yuri-xyz yuri-xyz self-assigned this Mar 21, 2025
@github-project-automation github-project-automation bot moved this to Not Started 🕧 in Tangle Universe Mar 21, 2025
@netlify
Copy link

netlify bot commented Mar 21, 2025

Deploy Preview for tangle-dapp ready!

Name Link
🔨 Latest commit b9d5350
🔍 Latest deploy log https://app.netlify.com/sites/tangle-dapp/deploys/67ecabdf7418890008cd298e
😎 Deploy Preview https://deploy-preview-2918--tangle-dapp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Mar 21, 2025

Deploy Preview for tangle-cloud ready!

Name Link
🔨 Latest commit b9d5350
🔍 Latest deploy log https://app.netlify.com/sites/tangle-cloud/deploys/67ecabdff76ad2000824c77d
😎 Deploy Preview https://deploy-preview-2918--tangle-cloud.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@netlify
Copy link

netlify bot commented Mar 21, 2025

Deploy Preview for tangle-leaderboard canceled.

Name Link
🔨 Latest commit 622fbb0
🔍 Latest deploy log https://app.netlify.com/sites/tangle-leaderboard/deploys/67e376f27bf5e8000875ad0a

@yuri-xyz yuri-xyz moved this from Not Started 🕧 to Building 🏗️ in Tangle Universe Mar 21, 2025
@yuri-xyz yuri-xyz changed the title feat(tangle-cloud): Implement operator transactions (approve, reject, unregister, etc.) feat(tangle-cloud): Implement additional operator transactions (unregister, terminate, etc.) Mar 21, 2025
@yuri-xyz
Copy link
Contributor Author

yuri-xyz commented Mar 21, 2025

@danielbui12 I just wrote the approve and reject tx's if you check the Files changed tab, you can use that to guide your implementation on your active PR. I'll be removing those and implementing the additional/missing ones instead so that you can go about approve & reject. In other words, feel free to copy paste it into your PR; I'll be removing it from this one to prevent conflicts.

@yuri-xyz yuri-xyz changed the title feat(tangle-cloud): Implement additional operator transactions (unregister, terminate, etc.) feat(tangle-cloud): Implement additional operator transactions (unregister, submitResult, etc.) Mar 21, 2025
@yuri-xyz yuri-xyz changed the title feat(tangle-cloud): Implement additional operator transactions (unregister, submitResult, etc.) feat(tangle-cloud): Implement additional services txs (unregister, request, etc.) Mar 21, 2025
@yuri-xyz yuri-xyz changed the title feat(tangle-cloud): Implement additional services txs (unregister, request, etc.) feat(tangle-cloud): Implement blueprint instance deployment page Mar 21, 2025
@drewstone
Copy link
Contributor

@yurixander do you plan to finish this soon or should we hand it to @danielbui12 ?

@yuri-xyz yuri-xyz assigned danielbui12 and unassigned yuri-xyz Mar 25, 2025
@danielbui12 danielbui12 force-pushed the yuri/create-services-txs branch from e709311 to c01c40c Compare March 26, 2025 01:20
@netlify
Copy link

netlify bot commented Mar 28, 2025

Deploy Preview for tangle-leaderboard ready!

Name Link
🔨 Latest commit b9d5350
🔍 Latest deploy log https://app.netlify.com/sites/tangle-leaderboard/deploys/67ecabdfd21eee00086fd1d3
😎 Deploy Preview https://deploy-preview-2918--tangle-leaderboard.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@danielbui12 danielbui12 marked this pull request as ready for review March 30, 2025 01:25
@danielbui12
Copy link
Contributor

need to merge #2937 first

onSave={(params) => {
tokenSymbol={network.tokenSymbol}
amountValue={amount[blueprint.id] ?? ''}
paramsValue={registrationParams[blueprint.id] ?? {}}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Might be good to put this on a useMemo

Copy link
Contributor

Choose a reason for hiding this comment

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

This is in map to render component, not a specific value. I tried useMemo got a bug, then I have to unwrap the useMemo

@drewstone
Copy link
Contributor

Why is the diff so large? Package lock?

@danielbui12 danielbui12 force-pushed the yuri/create-services-txs branch from 6451293 to 3eacf86 Compare April 1, 2025 08:24
@danielbui12 danielbui12 added needs review 👓 and removed on hold ⛔️ Indicates temporarily paused issues. labels Apr 1, 2025
};
});

return data as PrimitiveField[];
Copy link
Contributor

Choose a reason for hiding this comment

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

@yuri-xyz The type is too malicious to be covered so it's better to cast here

@danielbui12 danielbui12 merged commit 58c73c4 into develop Apr 3, 2025
20 checks passed
@danielbui12 danielbui12 deleted the yuri/create-services-txs branch April 3, 2025 01:55
@github-project-automation github-project-automation bot moved this from Building 🏗️ to Completed ✅ in Tangle Universe Apr 3, 2025
danielbui12 added a commit that referenced this pull request Apr 3, 2025
Co-authored-by: danielbui12 <[email protected]>
Co-authored-by: yurixander <[email protected]>
Co-authored-by: Trung-Tin Pham <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Completed ✅

Development

Successfully merging this pull request may close these issues.

[FEAT] Blueprint Instance Deployment Settings/ Configurations

5 participants