blob: 7112cec98bcd73b075bf3669078f17b8bd73c88d (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
|
name: SQL fix
description: Suggest a SQL only fix.
labels: [Comp-Database, Feedback-PatchFix]
body:
- type: markdown
attributes:
value: |
Thanks for taking the time to fill out this form! Please make sure to fill all fields, including the Title above.
For SQL fixes:
- search for existing issues, if a bug report exist for the issue you are trying to fix and it's opened post there.
- don't modify WDB fields without supply one sniff of those fields (censure guids, character names, anything blizzard can use to identify source)
- don't use pull requests for sql only fixes, unless it's to fix one existing unable to apply file.
- don't hardcode guids.
- don't DELETE + INSERT to update only few fields of one row.
- don't use database names.
- don't put ' around numbers.
- don't put ( ) if they aren't needed.
- use only 1 DELETE + INSERT when we add multiple items to one table, unless you are scripting multiple creatures.
- start fix by -- in case some previous sql misses proper ending.
- DELETE by guid AND entry to be sure we don't delete existing spawns.
The wiki page https://github.com/TrinityCore/TrinityCore/wiki/SQL-Fix includes additional details about how to fill this form and an example of a SQL fix.
- type: textarea
id: description
attributes:
label: Description
description: |
Description of what is getting fixed, including how to reproduce the issue if any.
validations:
required: true
- type: textarea
id: fix
attributes:
label: SQL fix
description: This field will be formatted automatically as SQL code block, no need to include \`\`\`sql
placeholder: |
--
render: sql
validations:
required: true
- type: dropdown
id: branch
attributes:
label: Branch
options:
- 3.3.5
- master
validations:
required: true
- type: textarea
id: commit
attributes:
label: TC rev. hash/commit
description: |
Copy the result of server debug command (if you need to run it from client get prat addon)
validations:
required: true
|
