LBRYCommunity/lbry-desktop

(quickfix) Now displays quit button in IncompatibleDaemonModal. #411

Merged

hackrush01 merged 1 commit from quickfix into master

2017-08-01 15:23:58 +02:00

hackrush01 commented

2017-07-31 16:31:26 +02:00

(Migrated from github.com)

Based on discussion I think this is what was asked. But since the earlier PR was closed so I added a new one.

This one displays a Learn More link pointing to the appropriate faq.

Based on discussion I think this is what was asked. But since the earlier PR was closed so I added a new one. This one displays a Learn More link pointing to the appropriate faq.

kauffj (Migrated from github.com) reviewed 2017-07-31 16:31:26 +02:00

hackrush01 (Migrated from github.com) reviewed 2017-07-31 17:13:07 +02:00

ui/js/component/modalIncompatibleDaemon/index.js

					
				@ -0,0 +1,12 @@

				import React from "react";

				import { connect } from "react-redux";

				import { doQuit, doSkipWrongDaemonNotice } from "actions/app";

hackrush01 (Migrated from github.com) commented

2017-07-31 17:13:07 +02:00

Is this doSkipWrongDaemonNotice required to be here? I could not find the corresponding command in actions/app.

Is this `doSkipWrongDaemonNotice` required to be here? I could not find the corresponding command in `actions/app`.

Sign in to join this conversation.

No reviewers

No labels

area: discovery

area: installer

area: livestream

area: performance

area: subscriptions

feature request

first-timers-only

good first issue

merge when green

needs: exploration

needs: grooming

needs: priority

needs: tech design

priority: blocker

priority: medium

protocol dependent

type: discussion

type: improvement

type: new feature

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#411

No description provided.