OWASP-BLT / BLT

OWASP BLT is a bug logging tool to report issues and get points, organizations are held accountable.
https://blt.owasp.org
GNU Affero General Public License v3.0
134 stars 133 forks source link

fix company account creating issue #2382

Closed JisanAR03 closed 2 months ago

JisanAR03 commented 3 months ago

for make it work we have to run "python manage.py migrate"

fixed #2380

JisanAR03 commented 3 months ago

I don't understand why we have a UU ID

@DonnieBLT , basically the previous code store uuid in company_id .that's why I also use uuid . if uuid is not needed then I will fix this error by using primary key

DonnieBLT commented 3 months ago

Thanks, yes UUID is not needed. We can remove it.

JisanAR03 commented 3 months ago

@DonnieBLT , @arkid15r . maybe we should have use company_id with a uuid . cause as basis of screenshot. there is already a function which make a uuid for each data , and our production application have few company so how can we remove the uuid without losing the company data? the code is from : website/models.py file

image

AtmegaBuzz commented 3 months ago

company_id was included during prev gsoc and somebody removed it causing the company dashboard to fail, i've used company_id in url so @JisanAR03 you have to fix the part where we are making the model change and separating the company id into another model. Company id should be unique and will shown in url while accessing companies and UUID is best we can use @DonnieBLT, if you want we can completely remove company_id field but then we have to use "primary key id" to access the company using url.

DonnieBLT commented 3 months ago

Let's use a slug of the company name as the unique identifier then. Those are more friendly urls.