Feature Request: Open Repo in browser #32

6543 · 2019-09-04T14:12:43Z

6543 commented

2019-09-04 14:12:43 +00:00

Issue: mmarif/GitNex#26

Issue: https://gitea.com/mmarif/GitNex/issues/26

mmarif commented

2019-09-05 10:14:28 +00:00

Thanks for the PR.

Adding it to the bottomsheet in repo detail is one way of dong it.

Another one which I am currently exploring(wip) is to add it to repo recyclerview with new menu which will have watchers, stars for the repo too.

But I will accept this PR as is to provide more flexibility.

Thanks for the PR. Adding it to the bottomsheet in repo detail is one way of dong it. Another one which I am currently exploring(wip) is to add it to repo recyclerview with new menu which will have watchers, stars for the repo too. But I will accept this PR as is to provide more flexibility.

👍 1

mmarif added the

Feature

Major-release

labels 2019-09-05 10:14:55 +00:00

mmarif added this to the 2.0.0 milestone 2019-09-05 10:15:09 +00:00

6543 commented

2019-09-05 22:54:21 +00:00

@mmarif where can i read how bmListener contect event handler with a function?

83e3564c13 is a try but i dont get it jet ...

@mmarif where can i read how bmListener contect event handler with a function? https://gitea.com/6543/GitNex/commit/83e3564c136d4ee1e947c11951680aff3b834526 is a try but i dont get it jet ...

👍 1

mmarif commented

2019-09-09 19:51:40 +00:00

Thank you @6543. I will check it out.

mmarif commented

2019-09-10 16:57:17 +00:00

To make this PR work, here are the working updates.

You can safely remove OpenWebRepoActivity.java activity file.

Add the following in RepoDetailActivity.java activity file in onButtonClicked function.

case "openWebRepo":
                TinyDB tinyDb = new TinyDB(getApplicationContext());
                String repoFullName = tinyDb.getString("repoFullName");
                String instanceUrlWithProtocol = "https://" + tinyDb.getString("instanceUrlRaw");
                if(!tinyDb.getString("instanceUrlWithProtocol").isEmpty()) {
                    instanceUrlWithProtocol = tinyDb.getString("instanceUrlWithProtocol");
                }
                Uri url = Uri.parse(instanceUrlWithProtocol + "/" + repoFullName);
                Intent i = new Intent(Intent.ACTION_VIEW, url);
                startActivity(i);
                break;

And in LoginActivity.java file,

String instanceUrlWithProtocol;
        if(protocol.toLowerCase().equals("https")) {
            instanceUrl = "https://" + instanceHost + "/api/v1/";
            instanceUrlWithProtocol = "https://" + instanceHost;
        }
        else {
            instanceUrl = "http://" + instanceHost + "/api/v1/";
            instanceUrlWithProtocol = "http://" + instanceHost;
        }

        tinyDb.putString("instanceUrlWithProtocol", instanceUrlWithProtocol);

Do let me know if you need any help with it.

To make this PR work, here are the working updates. You can safely remove `OpenWebRepoActivity.java` activity file. Add the following in `RepoDetailActivity.java` activity file in `onButtonClicked` function. ``` case "openWebRepo": TinyDB tinyDb = new TinyDB(getApplicationContext()); String repoFullName = tinyDb.getString("repoFullName"); String instanceUrlWithProtocol = "https://" + tinyDb.getString("instanceUrlRaw"); if(!tinyDb.getString("instanceUrlWithProtocol").isEmpty()) { instanceUrlWithProtocol = tinyDb.getString("instanceUrlWithProtocol"); } Uri url = Uri.parse(instanceUrlWithProtocol + "/" + repoFullName); Intent i = new Intent(Intent.ACTION_VIEW, url); startActivity(i); break; ``` And in `LoginActivity.java` file, ``` String instanceUrlWithProtocol; if(protocol.toLowerCase().equals("https")) { instanceUrl = "https://" + instanceHost + "/api/v1/"; instanceUrlWithProtocol = "https://" + instanceHost; } else { instanceUrl = "http://" + instanceHost + "/api/v1/"; instanceUrlWithProtocol = "http://" + instanceHost; } tinyDb.putString("instanceUrlWithProtocol", instanceUrlWithProtocol); ``` Do let me know if you need any help with it.

👍 1

6543 commented

2019-09-10 17:00:48 +00:00

i got the message twice is this a bug of gitea?

-> thanks, couldn't find the right locations where i have to change it. I'll try and push :D

i got the message twice is this a bug of gitea? -> thanks, couldn't find the right locations where i have to change it. I'll try and push :D

mmarif commented

2019-09-10 17:07:09 +00:00

i got the message twice is this a bug of gitea?

gitea.com sometime has "content corruption error", that happened just now. So I refreshed the page, that's why two messages.

This will help.

https://gitea.com/mmarif/GitNex/src/branch/master/app/src/main/java/org/mian/gitnex/activities/LoginActivity.java#L196

https://gitea.com/mmarif/GitNex/src/branch/master/app/src/main/java/org/mian/gitnex/activities/RepoDetailActivity.java#L155

> i got the message twice is this a bug of gitea? gitea.com sometime has "content corruption error", that happened just now. So I refreshed the page, that's why two messages. This will help. https://gitea.com/mmarif/GitNex/src/branch/master/app/src/main/java/org/mian/gitnex/activities/LoginActivity.java#L196 https://gitea.com/mmarif/GitNex/src/branch/master/app/src/main/java/org/mian/gitnex/activities/RepoDetailActivity.java#L155

mmarif reviewed 2019-09-10 17:21:09 +00:00

Dismissed

app/src/main/res/values/strings.xml Outdated