ovr / ghubber

:iphone: Mobile client for GitHub build on React Native :atom:
Other
51 stars 26 forks source link

Organization screen #47

Open ovr opened 7 years ago

ovr commented 7 years ago
  1. Use GraphQL API to fetch organization with pinned repositories (first: 10)
  2. Implement new component RepositoryBlock (this block will be used inside Profile screen and Organization screen):
  3. Implement PinnedRepositories component, this will render RepositoryBlock's (We will reuse this block on Profile)

Due GitHub limitations, only six repositories can be pinned inside organizations or user profile

<PinnedRepositories repositories={organization.repositories}>
</PinnedRepositories>

This will help us to build grid, there are different dimmensions on mobile devices for example on some tablets we can build 3x2 grid, on mobiles we can do 1x6

3x2:

|X|X|X| |X|X|X|

2x3

|X|X| |X|X| |X|X|

1x6:

|X| |X| |X| |X| |X| |X|

image

  1. Implement OrganizationScreen
  2. Implement navigation click on Profile organizations and SideMenu organizations

On organization page, There are two tabs:

image

  1. Overview:

image

  1. Repositories

Similar to profile Reposiotires

image

lex111 commented 7 years ago

Your token has not been granted the required scopes to execute this query. The 'name' field requires one of the following scopes: ['read:org'], but your token has only been granted the: ['user', 'public_repo', 'repo', 'notifications'] scopes. Please modify your token's scopes at: https://github.com/settings/tokens.

You need to change the initial permissions.

How best to implement an responsive grid? Use an external library for this?

ovr commented 7 years ago

How best to implement an responsive grid? Use an external library for this?

I dont know any external library, do known?

You need to change the initial permissions.

for plain login, you can add

https://github.com/ovr/ghubber/blob/master/actions/login.js#L84

About oauth, we have admin:org as I known

lex111 commented 7 years ago

Вот нужно текущую scope расширить, добавить туда read:org, иначе будет ошибка. Я это сделал вручную, на странице токенов https://github.com/settings/tokens

По библиотеке, я нашёл парочку, а-ля бутстрап :) вроде приколько, но не тестил, как это вообще реализуется, а то я думаю в терминах CSS

ovr commented 7 years ago

Вот нужно текущую scope расширить, добавить туда read:org, иначе будет ошибка. Я это сделал вручную, на странице токенов https://github.com/settings/tokens

Personal access tokens, это ведь для авторизации по логину или паролю - ты это можешь сам добавить в https://github.com/ovr/ghubber/blob/master/actions/login.js#L84

не работает разве так?

lex111 commented 7 years ago

@ovr я имею в виду, старые токены, помнишь, ты писал о миграции, типа юзер уже вошёл, а мы хоп, и расшили скоп, и нужно запросить заново токен..

ovr commented 7 years ago

@lex111 Об этом пока проще забыть сейчас, прилага то в маркете еще не была даже))

lex111 commented 7 years ago

Т.е. я хотел в пределах старого токена, а не новый создавать, это имею в виду, ведь просто добавление read:org не поможет, нужно повторно авторизоваться?

ovr commented 7 years ago

@lex111

добавление read:org не поможет, нужно повторно авторизоваться?

Да, нужно переавторизоваться!

lex111 commented 7 years ago

Ну вот, и нужен механизм проверки токена, API это позволяет, и если нужно перегенерировать новый токен, то выходить из аккаунта, и просить заново авторизовать?

ovr commented 7 years ago

Ну вот, и нужен механизм проверки токена, API это позволяет, и если нужно перегенерировать новый токен, то выходить из аккаунта, и просить заново авторизовать?

image

Лучше спрашивать сейчас сразу права на огранизации!

lex111 commented 7 years ago
Оффтоп Не знаю что ты не спишь, но заодно может посмотришь мой древний PR :rofl: https://github.com/ovr/ghubber/pull/18#issuecomment-327507168 ? Там уже правда образовались конфликты )
lex111 commented 7 years ago

Ну я вперёд смотрю, типа сейчас это сделать