Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Custom Caleuche Support #26

Open
ajunge opened this issue Nov 19, 2018 · 29 comments
Open

Custom Caleuche Support #26

ajunge opened this issue Nov 19, 2018 · 29 comments

Comments

@ajunge
Copy link

ajunge commented Nov 19, 2018

Motivation: Right now the mobile app only allows to backup to uport hosted caleuche service at https://api.uport.me/caleuche. Some users want to run their own instance of caleuche, so we need a way on the mobile app to add the custom caleuche URL and start backing up there.

Work needed: We need to add a screen on the "Account Back Up" option, that allows the user to add a custom URL for their own instance of caleuche. Also need to change the functionality to use the new url ( https://github.com/uport-project/uport-mobile/blob/02db5e6f5d70073b8f982eda4946b071caa4315f/lib/sagas/hubSaga.js ). If the user is changing from any backup service, the new service needs to be updated with the latest data.

Acceptance criteria: A runnable version of the mobile app (based out of develop branch) that allows to change the backup server and migrate all the already backed up data to the new service.

Please feel free to ask any question in case something is not well explained

@ajunge
Copy link
Author

ajunge commented Nov 19, 2018

To run your own instance of caleuche: https://github.com/uport-project/lambda-caleuche/blob/develop/RUN_INSTANCE.md

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


This issue now has a funding of 250.0 DAI (250.0 USD @ $1.0/DAI) attached to it as part of the ConsenSys / uPort fund.

@tpscrpt
Copy link

tpscrpt commented Nov 19, 2018

I'll give it a shot. Not too confident about the UI, but it might work out!

@ajunge
Copy link
Author

ajunge commented Nov 19, 2018

Let us know if you run into any problems

@gitcoinbot
Copy link

gitcoinbot commented Nov 20, 2018

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work has been started.

These users each claimed they can complete the work by 2 weeks, 6 days ago.
Please review their action plans below:

1) jamesdigid has been approved to start work.

First I'll look over the existing "Account Backup Option," and get familiar with the current functionality. I'll need to review data manipulation functionality already written, along with network service operations, and any REST API's to understand the flow on how data is backed up already. After on-boarding i'll build out the functionality such that a custom url is entered and data is either both backed up and/or migrated there.

Learn more on the Gitcoin Issue Details page.

2) ramanandapanda has been approved to start work.

I would like to work on it. I have worked on react native.
what are the initial steps to setup the dev env?
Please guide.
I cannot clone git clone [email protected]:uport-project/lambda-caleuche.git getting permission denied.

Learn more on the Gitcoin Issue Details page.

@ajunge
Copy link
Author

ajunge commented Nov 20, 2018

I cannot clone git clone [email protected]:uport-project/lambda-caleuche.git getting permission denied.

Weird. Is public repo.
Trying to reproduce it

@rmw2
Copy link

rmw2 commented Nov 20, 2018

It's my understanding that we do require public key auth on most of our repos -- if there is no ssh key on your github account, it won't let you push/pull/clone

@jamesdigid
Copy link

jamesdigid commented Nov 20, 2018

@ajunge
--I had some questions about the custom url they would be inputting. Such as do you plan to support mostly AWS storage services like s3, or others such as ftp, sftp, etc. --

Just finished setting up the lambda-caleuche i was misunderstanding what the custom url was.

Thanks

@ajunge
Copy link
Author

ajunge commented Nov 20, 2018

@jamesdigid You can work on your solution too. We don't know if the other person will finish, and if both come out with a solution then we can find a way make everyone happy.
The customUrl is always an https://...... caleuche instance. If not, is an invalid url.
In the future we expect to write more storage modules for caleuche to provide more alternatives to our users.

@jamesdigid
Copy link

@ajunge So just to clarify you want this feature built off of develop branch? I'm running into a few issues with the delete functionality and just want make sure that develop is stable enough to work off of? Because i'm getting inconsistent results. Originally the develop branch had a functional account backup with delete. Now i seem to be stuck in a delete state.

I've followed through the logic and found the saga flow handling this, it seems to be returning a 500 error with: "index.map is not a function" as the error message. I found this also exist on my local fork'ed copy but still hitting same api end point. Is there any reason we aren't branching from master for the pull request?

@jasheal
Copy link

jasheal commented Nov 21, 2018

@jamesdigid we branch from develop and make pull requests back there. Master is the release branch. I looked into your issue and it appear that doing a backup with remote debugger on corrupts the backup so it can't be deleted. Rebuild new app and try without remote debug?

@gitcoinbot
Copy link

@ramanandapanda Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@gitcoinbot
Copy link

@jamesdigid Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@gitcoinbot
Copy link

@ramanandapanda Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@ocio28
Copy link

ocio28 commented Nov 28, 2018

where to store the url, in the device (asyncstorage)? or other way?

@ajunge
Copy link
Author

ajunge commented Nov 28, 2018

I think is stored on the redux state. But I prefer confirmation from @rmw2

@jamesdigid
Copy link

@ajunge Commenting to back, to say restarting work on this issue.

@jamesdigid
Copy link

jamesdigid commented Nov 29, 2018

@jasonhealy I attempted to delete with live debug turned off, is there any other local cache that I
should know to get back to a working state?

@jasheal
Copy link

jasheal commented Nov 29, 2018

@jamesdigid Delete the app from device/simulator and rebuild. Save the url to the settings key in redux which gets persisted.

@ocio28
Copy link

ocio28 commented Nov 29, 2018

i try to configure a caleuche instance following the instructions but the encrypt command do nothing (no error, no file change)

sls encrypt -n SECRETS:BUCKET -v caleuche -k 93589337-f911-4379-983d-694...

@gitcoinbot
Copy link

@ramanandapanda Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@gitcoinbot
Copy link

@jamesdigid Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@gitcoinbot
Copy link

@ramanandapanda Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

@jamesdigid
Copy link

jamesdigid commented Dec 3, 2018

@jamesdigid Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • reminder (3 days)
  • escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

Commenting to say I submitted pull request for this feature. Let me know if this works.

@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 250.0 DAI (250.0 USD @ $1.0/DAI) has been submitted by:

  1. @jamesdigid

@ajunge please take a look at the submitted work:


@gitcoinbot
Copy link

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


Work for 250.0 DAI (250.0 USD @ $1.0/DAI) has been submitted by:

  1. @jamesdigid

@ajunge please take a look at the submitted work:


@CPSTL
Copy link

CPSTL commented Dec 10, 2018

Hey @ajunge ! Have you had the chance to take a look at the submitted work yet?

@ajunge
Copy link
Author

ajunge commented Dec 12, 2018

@jamesdigid did you see the review comment on the PR #38 ?

@jamesdigid
Copy link

jamesdigid commented Feb 15, 2019

@ajunge
Yes, I spent time developing this solution and made mention of a previously existing issue and was still denied my bounty... This is a waste of time if you're QA people are not willing to actually test my PR.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

8 participants