Comments (14)
@chhetri28 @whoabuddy I requested to work on this and was assigned it and did a lot of work to figure it out, I should probably be the person submitting the pull request unless somebody says otherwise. Another pull request was made randomly following information that was posted on here answering my questions. I'm trying to get experience and proof of experience. It's not a big deal but I should get credit for doing the work in the form of hopefully an accepted pull request.
from ui.
I'm trying to get experience and found this on an aggregator listing good first issue tags. I can work on this if you're accepting contributors.
from ui.
Sure, that'd be great! Basic process would be to:
- fork the repository
- create a branch for the changes
- submit a PR back with the changes
Happy to help with any questions along the way.
If you're on Discord you can join the CityCoins server as well - there are a few areas for builders.
from ui.
@whoabuddy Awesome, thank you, I'll get it in later today!!
from ui.
@whoabuddy sorry about the delay, something came up yesterday. I'm working on it now. So when I search in the main branch of the unforked repository for "substr()" in the search bar, it brings up instances existing in the branch 1a620c0. When I search for "substr()" in my fork, it doesn't find any results. Should I have forked branch 1a620c0 instead of the main branch?
from ui.
You should checkout the develop
branch and create the PR from there, it's the main one that has the most recent code, and once enough changes are finalized it's released to main
.
As far as the search I think it's a simple fix: substr()
won't match anything because the commands contain values between (
and )
, e.g. {addr.substr(0, 5)}...{addr.substr(addr.length - 5)}
. If you search for just substr
instead you'll see the components that use it:
https://github.com/citycoins/ui/search?q=substr
from ui.
@phershbe are you still having trouble with this? If you are, I can give it a try. Thanks.
from ui.
@randr000 I'm still working on it, I don't like giving up on things. I'm still trying to figure out why substr won't show up in the search on my fork.
from ui.
@whoabuddy that's good to know about searching for substr
and not substr()
, that's a stupid mistake and it's good to learn from. It still finds substr()
, I probably searched both ways before too. In any case, I'm still having problems.
I've worked a little bit on some other open source projects and have never had this problem; there seems to be something really simple that I am not understanding in this case.
Here are some screenshots:
Searching in the unforked repository:
The fork that I created:
Searching in my forked repository:
from ui.
One option is to delete the fork and start over - sometimes something funny happens between and a fresh start eliminates any weirdness.
If I couldn't find it via the search then I'd look for those components in their directories and see if the code matches what I expect to see, e.g. https://github.com/citycoins/ui/blob/develop/src/components/profile/Address.js#L11-L13
Does that component/folder structure look the same as your fork? If not, you may be on the wrong branch?
from ui.
@whoabuddy Thank you for your patience, I'm learning a lot here and hopefully can get this resolved quickly.
Deleting the fork and starting over didn't work.
So in the search for substr
in my fork, the issues
tab was selected in the search. When I changed the tab to code
, it gave me "Sorry, forked repositories are not currently searchable.". Okay so I searched substr
again in the unforked repository and it found five files:
src/components/Address.js
src/components/Tx.js
src/components/TxStatus.js
src/lib/transactions.js
src/components/CityCoinTxList.js
Clicking on them, however, shows me files in the 10620c0596 branch. I then searched for them in the develop
branch by clicking through the components
and lib
folders and couldn't find most of them (Tx.js, TxStatus.js, transactions.js, CityCoinTxList.js) anywhere except for the Address.js file that you mentioned in the previous comment. My logic there was that if I could find them by clicking through folders, I could still go in and change them in my fork.
Am I correct in my reasoning and results above?
from ui.
@phershbe No problem! I still suspect what's happening here is related to the default branch main
vs develop
where the most recent changes live. The paths you have listed above are linked to commit 1a620c0, which is definitely a much older one.
The search also isn't working as expected for me, I ended up using the search in VS Code to find which files: it's just the LinkTx.js and Address.js components.
Depending on how you're accessing things (website, command-line, etc) there are a few steps to make sure you see the right files. Here are links to the two components on the develop branch:
https://github.com/phershbe/ui/blob/develop/src/components/profile/Address.js
https://github.com/phershbe/ui/blob/develop/src/components/common/LinkTx.js
You can also sync your fork beforehand as it's two commits behind.
from ui.
Hey @whoabuddy I've replaced the substr() method with substring() on both of the file.
I'm new to this, could you please tell what's the next step?
from ui.
@whoabuddy Awesome, thank you for your help and the detail in answering my questions. Yeah I kind of thought about searching in Visual Studio Code before, I didn't think it would give me different results or something, that's great advice though.
These kinds of challenges are good because I learn a lot from them, it took some time and effort from you though to help so thank you.
I'm going to browse through the issues and see what else I could help with.
from ui.
Related Issues (20)
- Use CHAIN_SUFFIX where possible HOT 1
- 🐞[BUG] Claim stacking reward state overwritten
- Smarter data fetching for the dashboard
- Add STX/MIA ratio to dashboard mining stats
- ⚡[FEAT] Detect stale data
- 🐞[BUG] Mining and stacking tabs do no load
- 🐞[BUG] mining claim state wrong if API call fails
- ⚡[FEAT] HOT 1
- ⚡[FEAT] The staked New York Coin did not come into my wallet HOT 1
- 🐞[BUG] Finished stacking cycles show pending
- 🐞[BUG] Claim Mining Reward
- ⚡[FEAT] Aggregate Many Websites into Main Site
- 🐞[BUG] Can't connect to Hiro Wallet
- 🐞[BUG] Can't claim rewards/stacked coins HOT 1
- 🐞[BUG] Claiming MIA keeps getting rolled back by post-condition HOT 5
- 🐞[BUG] Unable to load user balance HOT 1
- Notice This transaction would have succeeded, but was rolled back by a supplied post-condition. ⚡[FEAT] HOT 2
- Notice This transaction would have succeeded, but was rolled back by a supplied post-condition.🐞[BUG] HOT 2
- 🐞[BUG] Unable to load current reward cycle HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ui.