LBRYCommunity/lbry-desktop
1
0
Fork 0
Code Issues 596 Pull requests 6 Projects Releases 238 Packages Wiki Activity

Merge pull request #1652 from lbryio/subscriptions-fix

Browse source 
fix: fetch subscriptions if there are no claims loaded
This commit is contained in:
Sean Yesmunt 2018-06-20 15:11:40 -04:00 committed by GitHub
commit 249a0c2844
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 13 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
src/renderer/page/subscriptions/view.jsx
View file

@ -6,6 +6,7 @@ import * as NOTIFICATION_TYPES from 'constants/notification_types';
import Button from 'component/button';
import FileList from 'component/fileList';
import type { Claim } from 'types/claim';
import isDev from 'electron-is-dev';
type Props = {
doFetchClaimsByChannel: (string, number) => void,
@ -42,7 +43,18 @@ export default class extends React.PureComponent<Props> {
const subscriptionClaimMap = {};
subscriptionClaims.forEach(claim => {
subscriptionClaimMap[claim.uri] = 1;
/*
This check added 6/20/18 to fix function receiving empty claims unexpectedly.
The better fix is ensuring channels aren't added to byId if there are no associated claims
We are adding this now with the redesign release to ensure users see the correct subscriptions
*/
if (claim.claims.length) {
subscriptionClaimMap[claim.uri] = 1;
} else if (isDev) {
console.error(
`claim for ${claim.uri} was added to byId in redux but there are no loaded fetched claims`
);
}
});
subscriptions.forEach(sub => {