Skip to content
This repository has been archived by the owner on Apr 25, 2023. It is now read-only.

feat: public settings page #32

Merged
merged 16 commits into from
Jul 16, 2021
Merged

feat: public settings page #32

merged 16 commits into from
Jul 16, 2021

Conversation

KaWaite
Copy link
Member

@KaWaite KaWaite commented Jul 14, 2021

Overview

Main idea is to split project settings into project and public.

What I've done

  • split project settings into project and public settings pages
  • update language
  • better hide when archived (public, dataset and plugin sections will be hidden as well now)
  • dataset section bugs fixed (adding and deleting)

What I haven't done

How I tested

Screenshot

Screen Shot 2021-07-15 at 13 23 09

Screen Shot 2021-07-15 at 13 23 21

Which point I want you to review particularly

Memo

@codecov
Copy link

codecov bot commented Jul 14, 2021

Codecov Report

Merging #32 (305a242) into main (4cddcca) will increase coverage by 0.02%.
The diff coverage is 50.00%.

❗ Current head 305a242 differs from pull request most recent head 5dd1524. Consider uploading reports for the commit 5dd1524 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   52.15%   52.18%   +0.02%     
==========================================
  Files          45       45              
  Lines         650      642       -8     
  Branches      125      125              
==========================================
- Hits          339      335       -4     
+ Misses        237      233       -4     
  Partials       74       74              
Impacted Files Coverage Δ
src/state/reducer.ts 75.00% <ø> (ø)
src/gql/graphql-client-api.tsx 52.73% <50.00%> (+0.10%) ⬆️

@KaWaite KaWaite marked this pull request as ready for review July 15, 2021 05:48
HideBa
HideBa previously approved these changes Jul 15, 2021
Copy link
Member

@HideBa HideBa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@KaWaite KaWaite requested review from rot1024 and issmail-basel July 15, 2021 09:05
@rot1024 rot1024 force-pushed the main branch 2 times, most recently from 0752537 to 21b167f Compare July 16, 2021 04:02
@KaWaite KaWaite requested a review from rot1024 July 16, 2021 07:47
@HideBa HideBa merged commit ebfd416 into main Jul 16, 2021
@HideBa HideBa deleted the feat-public-settings branch July 16, 2021 08:30
keiya01 pushed a commit that referenced this pull request Apr 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants