-
Notifications
You must be signed in to change notification settings - Fork 43
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
Redo Carousel #196
Comments
/start |
Tips:
|
Still available? |
No, on it |
You should have at least a draft pull open to retain this task |
I will get one open soon, too many files to change.. wasn't a 1 day task as a lot would easily break |
# No linked pull requests to close |
@pavlovcik Since the list might get quite long isn't it necessary to implement some form of pagination here? Probably after 10 permits, we should break to the next page. It will improve the UX significantly in my opinion. |
Makes sense but I think in the near term let's not worry about that. The only planned behavior for multi permit payouts is something like 1. Stablecoin 2. Governance token reward 3. NFT POAP reward. So the most I expect to support is three for our near term vision. |
/start |
Tips:
|
@pavlovcik and @rndquu, I'm currently working on setting up a repository, and I've been following the instructions provided in the README. Although I managed to start the application, I'm encountering an issue where it continuously loads without rendering any actual data. Could you please offer some guidance on troubleshooting steps to get the application working as intended? Any insights or suggestions would be greatly appreciated. Thank you! |
In order for any data to be displayed the You may generate test claim URLs with this script. If the app is still loading continuously the issue is somewhere else. |
@rndquu I'm using the |
Try now #213 Be sure to pull |
Works now Thanks |
# These linked pull requests are closed: <a href="https://github.com/ubiquity/pay.ubq.fi/pull/216">#216</a> |
/start |
Tips:
|
Tips:
|
@pbkompasz, this task has been idle for a while. Please provide an update. |
@pbkompasz, this task has been idle for a while. Please provide an update. |
1 similar comment
@pbkompasz, this task has been idle for a while. Please provide an update. |
It looks like there haven't been any changes. Best if there are fixes made and if our continuous deployments work I can test from mobile to approve |
As far as I understand this PR should be reopened against the latest |
@pbkompasz, this task has been idle for a while. Please provide an update. |
You can reopen your pull like what rndquu said and we can proceed |
Is this still valid considering the mint card functionality? |
I raised this same point during review of |
@pbkompasz the deadline is at Wed, Oct 23, 7:23 AM UTC |
! Failed to run comment evaluation. SyntaxError: Unterminated string in JSON at position 35 |
1 similar comment
! Failed to run comment evaluation. SyntaxError: Unterminated string in JSON at position 35 |
@gentlementlegen there is a parser error and I think it would make sense to include the sourcemap because debugging from a compiled js file obviously is not the best. |
|
It also says |
I think the aesthetics are not good with the carousel.
Instead we should render all the permits vertically stacked all at once. Something like the following:
I suspect that we will need to redo quite a bit of logic, as the app seems to be designed only to render one at a time.
The text was updated successfully, but these errors were encountered: