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

chore(remove): cleanup of old files #4987

Merged
merged 4 commits into from
Sep 26, 2022
Merged

chore(remove): cleanup of old files #4987

merged 4 commits into from
Sep 26, 2022

Conversation

molimauro
Copy link
Contributor

@molimauro molimauro commented Sep 22, 2022

What this PR does 📖

  • title

Which issue(s) this PR fixes 🔨

Special notes for reviewers 🗒️

Additional comments 🎤

  • SolanaManager tests skipped

@github-actions github-actions bot added the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Sep 22, 2022
@molimauro
Copy link
Contributor Author

@iltumio can you please check if there are any Solana config we can remove and any functions in the SolanaManager we dont use anymore (like the requestAirdrop)? Thanks!

@stavares843 stavares843 added the draft A developer wants eyes on this PR, but they don't think it's ready to merge. label Sep 22, 2022
@molimauro molimauro changed the title chore(remove): cleanup of solana old files chore(remove): cleanup of old files Sep 23, 2022
@molimauro molimauro marked this pull request as ready for review September 23, 2022 16:03
@molimauro molimauro removed the draft A developer wants eyes on this PR, but they don't think it's ready to merge. label Sep 23, 2022
@github-actions github-actions bot added the missing fixing conflict Conflict needs to be handled, then re-tested by devs/qa label Sep 23, 2022
@stavares843 stavares843 removed the missing fixing conflict Conflict needs to be handled, then re-tested by devs/qa label Sep 23, 2022
@aewing aewing removed the Missing Dev Review A Dev and a Dev Approver need to review the PR, then mark as Ready for QA label Sep 23, 2022
@github-actions github-actions bot added the missing fixing conflict Conflict needs to be handled, then re-tested by devs/qa label Sep 26, 2022
@molimauro
Copy link
Contributor Author

Why this is blocked @stavares843?

@josephmcg
Copy link
Contributor

yeah this would be good to get in soon so we don't get more conflicts

@github-actions github-actions bot removed the missing fixing conflict Conflict needs to be handled, then re-tested by devs/qa label Sep 26, 2022
@stavares843
Copy link
Member

this fleek

Captura de ecrã 2022-09-26, às 14 48 30

vs dev

Captura de ecrã 2022-09-26, às 14 48 44

something broke with the modal 🔨

@molimauro
Copy link
Contributor Author

Sorry @stavares843 i dont have that issue locally 😅

@molimauro
Copy link
Contributor Author

molimauro commented Sep 26, 2022

Its not related to this, seems a problem with the nodes config on Fleek

@stavares843
Copy link
Member

will test locally instead 😅 thanks for taking a look

@molimauro
Copy link
Contributor Author

Gonna merge this!

@stavares843
Copy link
Member

🎉

@molimauro molimauro merged commit 2a4a93b into dev Sep 26, 2022
@molimauro molimauro deleted the solana-cleanup branch September 26, 2022 15:59
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.

Solana Cleanup
6 participants