-
Notifications
You must be signed in to change notification settings - Fork 144
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
New file inputs #1009
Comments
Hi, @Marchand-Nicolas |
I'd like to handle this task. |
Can I work on this, please? |
Could I take over this issue? |
Can I be assigned to this? |
@Marchand-Nicolas |
May I take care of this? |
Could I take on this issue? |
I would love to make this my first contribution on Starknet. Here's my approach to implementing the requested changes:
I'll add file input fields adjacent to the existing NFT Image Path and Issuer Logo text inputs without replacing them. The file inputs will be styled to align with the existing UI, ensuring the layout looks clean and consistent. Both the file inputs and text inputs will remain functional, with no server-side logic implemented at this stage.
I’ll submit the PR with the label “ready for review” once the changes are tested. The PR description will include Close #[issue_id] to ensure it links to the issue. My ETA is 8 hours tops. I’ll keep everything straightforward and follow the provided requirements to ensure a smooth review process. |
Description 📹
Update
components/admin/formSteps/RewardDetailsForm.tsx
to add a file input field next to the existingNFT Image Path
andIssuer Logo
text input fields. The new file input fields should not replace the current text inputs but should be cleanly styled alongside them. No logic implementation is needed for these fields yet; they will be used to upload an image to the server and fill the corresponding link fields automatically in another issue.Proposed Actions 🛠️
Here’s a checklist of actions to follow for resolving this issue:
Fork the repository and create a new branch using the issue number:
NFT Image Path
andIssuer Logo
text inputs.Code Snippet Example:
Make sure your changes don't break existing functionality and commit with a clear message:
git commit -m "Fix: Add file inputs for NFT Image and Issuer Logo"
Required 📋
To keep our workflow smooth, please make sure you follow these guidelines:
Thank you for your contribution 🙏
The text was updated successfully, but these errors were encountered: