tylerbenson / vyllage

https://www.vyllage.com
4 stars 0 forks source link

[Skill Section] Deleting the last skill actually deletes the previous one. #1226

Open UnspeakableHorror opened 8 years ago

UnspeakableHorror commented 8 years ago

1) Click Edit on the Skill section. 2) Add Skill Test1. 3) Save. 4) Click Edit on the Skill section. 5) Add Skill Test2. 5) Save. 6) Click Edit on the Skill section. 7) Delete Skill Test1. 8) Save.

Result: Test1 is deleted. Expected result Test2 to be deleted.

Edit: Deleting any of the previous to the last one results in deleting the last one instead when there are more than 2 skills.

UnspeakableHorror commented 8 years ago

@nelonoel Maybe this https://github.com/natebenson/vyllage/tree/tareq/tag-fix has something that fixes it?

UnspeakableHorror commented 8 years ago

There's a PR too, https://github.com/natebenson/vyllage/pull/1210.