lunny/gitea
Archived
1
1
Fork 1
Code Issues 1.7k Pull Requests 154 Projects Releases 128 Wiki Activity

Add default values for settings #455

Merged
lunny merged 4 commits from lunny/add_default_setting_values into master 2016-12-23 07:18:06 +00:00
Conversation 6 Commits 4 Files Changed 6 +154 -24
lunny commented 2016-12-23 02:40:44 +00:00
Owner
Copy Link
No description provided.
ethantkoenig commented 2016-12-23 04:53:44 +00:00 (Migrated from github.com)
Author
Owner
Copy Link

@lunny It looks like your build failed due to a test I added in #442. Sorry about that, there is a race condition and I will have a fix for it very soon. In the meantime, if you just rerun the build, the test will most likely succeed. Again, sorry for any inconvenience.

@lunny It looks like your build failed due to a test I added in #442. Sorry about that, there is a race condition and I will have a fix for it very soon. In the meantime, if you just rerun the build, the test will most likely succeed. Again, sorry for any inconvenience.
bkcsoft commented 2016-12-23 04:54:11 +00:00 (Migrated from github.com)
Author
Owner
Copy Link

@lunny I think you've missed a few keys. Like this one 230cabb4c4/modules/setting/setting.go (L759)

@lunny I think you've missed a few keys. Like this one https://github.com/lunny/gitea/blob/230cabb4c45a076f75244cc247ef98b226e2284f/modules/setting/setting.go#L759
lunny commented 2016-12-23 04:59:20 +00:00
Author
Owner
Copy Link

Yes. I will send more commits.

Yes. I will send more commits.
tboerger commented 2016-12-23 06:22:10 +00:00 (Migrated from github.com)
Author
Owner
Copy Link

LGTM

LGTM
tboerger (Migrated from github.com) reviewed 2016-12-23 06:32:11 +00:00
modules/setting/setting.go Outdated
tboerger (Migrated from github.com) commented 2016-12-23 06:31:46 +00:00
Author
Owner

This default worked well for me, I had the installer initially.

This default worked well for me, I had the installer initially.
modules/setting/setting.go Outdated
tboerger (Migrated from github.com) commented 2016-12-23 06:31:20 +00:00
Author
Owner

Really? This gets generated for me on the installer

Really? This gets generated for me on the installer
lunny reviewed 2016-12-23 06:40:04 +00:00
modules/setting/setting.go Outdated
lunny commented 2016-12-23 06:40:04 +00:00
Author
Owner

So, it's a good idea to give it a default value.

So, it's a good idea to give it a default value.
tboerger (Migrated from github.com) reviewed 2016-12-23 06:41:26 +00:00
modules/setting/setting.go Outdated
tboerger (Migrated from github.com) commented 2016-12-23 06:41:26 +00:00
Author
Owner

But does it still generate a random new string on install?

But does it still generate a random new string on install?
lunny reviewed 2016-12-23 06:46:29 +00:00
modules/setting/setting.go Outdated
lunny commented 2016-12-23 06:46:29 +00:00
Author
Owner

Yes. generate a random will always on installation and write back to ini file.

Yes. generate a random will always on installation and write back to ini file.
tboerger commented 2016-12-23 06:47:26 +00:00 (Migrated from github.com)
Author
Owner
Copy Link

Still LGTM

Still LGTM
bkcsoft commented 2016-12-23 07:17:28 +00:00 (Migrated from github.com)
Author
Owner
Copy Link

LGTM

LGTM
This repo is archived. You cannot comment on pull requests.
Reviewers
No reviewers
Labels
Clear labels   
backport/done

   
backport/v1.0

   
backport/v1.1

   
backport/v1.10

   
backport/v1.11

   
backport/v1.12

   
backport/v1.13

   
backport/v1.14

   
backport/v1.15

   
backport/v1.2

   
backport/v1.3

   
backport/v1.4

   
backport/v1.5

   
backport/v1.6

   
backport/v1.7

   
backport/v1.8

   
backport/v1.9

   
bounty

   
changelog

   
dependencies

Pull requests that update a dependency file

  
frontport/done

   
frontport/main

   
good first issue

Likely to be an easy fix

  
Hacktoberfest

   
hacktoberfest-accepted

   
in progress

   
kind/api

   
kind/breaking

   
kind/bug

   
kind/build

   
kind/deployment

   
kind/deprecated

   
kind/docs

   
kind/enhancement

   
kind/feature

   
kind/lint

   
kind/misc

   
kind/moderation

features for moderateing spam, abuse, ...

  
kind/package

   
kind/proposal

   
kind/question

   
kind/refactor

   
kind/regression

   
kind/security

   
kind/summary

   
kind/testing

   
kind/translation

   
kind/ui

   
kind/upstream-related

An issue that is caused by an upstream dependency

  
kind/usability

   
kind/ux

   
lgtm/done

   
lgtm/need 1

   
lgtm/need 2

   
performance/bigrepo

Performance Issues affecting Big Repositories

  
performance/cpu

   
performance/memory

Performance issues affecting memory use

  
performance/speed

performance issues with slow downs

  
priority/critical

   
priority/low

   
priority/maybe

   
priority/medium

   
proposal/rejected

   
reviewed/confirmed

Issue has been reviewed and confirmed to be present or accepted to be implemented

  
reviewed/duplicate

   
reviewed/fixed

A fix for this issue is already in the attached milestone

  
reviewed/invalid

   
reviewed/not-a-bug

The reported issue is the intended behavior

  
reviewed/wontfix

   
skip-changelog

   
stale

   
status/blocked

   
status/needs-feedback

   
status/wip

   
theme/2fa

   
theme/authentication

   
theme/avatar

   
theme/backup-restore

   
theme/docker

Docker themed issued

  
theme/federation

   
theme/issues

   
theme/kanban

   
theme/markdown

   
theme/migration

Migration issues

  
theme/mobile

   
theme/pr

   
theme/signing

   
theme/sqlite

   
theme/timetracker

   
theme/webhook

   
theme/wiki
No Label
backport/done
backport/v1.0
backport/v1.1
backport/v1.10
backport/v1.11
backport/v1.12
backport/v1.13
backport/v1.14
backport/v1.15
backport/v1.2
backport/v1.3
backport/v1.4
backport/v1.5
backport/v1.6
backport/v1.7
backport/v1.8
backport/v1.9
bounty
changelog
dependencies
frontport/done
frontport/main
good first issue
Hacktoberfest
hacktoberfest-accepted
in progress
kind/api
kind/breaking
kind/bug
kind/build
kind/deployment
kind/deprecated
kind/docs
kind/enhancement
kind/feature
kind/lint
kind/misc
kind/moderation
kind/package
kind/proposal
kind/question
kind/refactor
kind/regression
kind/security
kind/summary
kind/testing
kind/translation
kind/ui
kind/upstream-related
kind/usability
kind/ux
lgtm/done
lgtm/need 1
lgtm/need 2
performance/bigrepo
performance/cpu
performance/memory
performance/speed
priority/critical
priority/low
priority/maybe
priority/medium
proposal/rejected
reviewed/confirmed
reviewed/duplicate
reviewed/fixed
reviewed/invalid
reviewed/not-a-bug
reviewed/wontfix
skip-changelog
stale
status/blocked
status/needs-feedback
status/wip
theme/2fa
theme/authentication
theme/avatar
theme/backup-restore
theme/docker
theme/federation
theme/issues
theme/kanban
theme/markdown
theme/migration
theme/mobile
theme/pr
theme/signing
theme/sqlite
theme/timetracker
theme/webhook
theme/wiki
Milestone
Clear milestone
No items
No Milestone
1.0.0
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
1 Participants
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: lunny/gitea#455
No description provided.
Delete Branch "lunny/add_default_setting_values"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?