Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DPL-1056 - Removed breaking restriction #744

Merged
merged 2 commits into from
Apr 9, 2024

Conversation

SHIV5T3R
Copy link
Member

@SHIV5T3R SHIV5T3R commented Apr 8, 2024

Closes #701

Changes proposed in this pull request

  • Removed a restriction that would prevent new locations from being added if the same location type was being used

Instructions for Reviewers

[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

Copy link
Contributor

@dasunpubudumal dasunpubudumal left a comment

Choose a reason for hiding this comment

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

@SHIV5T3R the change looks good.

A small thing; the README.md doesn't have any instruction on setting up bunny.yml file. I imagine one has to go to the workflow (in .github directory) to find out how to utilise it. When I was first setting it up, I ran to an error saying the bunny.yml was not found.

Do you think that we could improve the README.md to include instructions on the bunny.yml file?

Copy link
Contributor

@KatyTaylor KatyTaylor left a comment

Choose a reason for hiding this comment

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

Looks good.

On the related story, Harry wrote:

After removing this, the behaviour is still the same as previously .... Another room with the same name cannot be created, because of the validation validates :name, presence: true, uniqueness: { scope: :ancestry, case_sensitive: true } in the location class, but another location type with the same name can still be created.

(#649 (comment))

I assume that's still true here.

@KatyTaylor
Copy link
Contributor

@SHIV5T3R the change looks good.

A small thing; the README.md doesn't have any instruction on setting up bunny.yml file. I imagine one has to go to the workflow (in .github directory) to find out how to utilise it. When I was first setting it up, I ran to an error saying the bunny.yml was not found.

Do you think that we could improve the README.md to include instructions on the bunny.yml file?

I've done this in a separate PR :) #745

@dasunpubudumal
Copy link
Contributor

@SHIV5T3R the change looks good.

A small thing; the README.md doesn't have any instruction on setting up bunny.yml file. I imagine one has to go to the workflow (in .github directory) to find out how to utilise it. When I was first setting it up, I ran to an error saying the bunny.yml was not found.

Do you think that we could improve the README.md to include instructions on the bunny.yml file?

Disregard this. Just saw Katy had updated the README. All good!

@SHIV5T3R SHIV5T3R merged commit 93c2e7c into develop Apr 9, 2024
4 checks passed
@SHIV5T3R SHIV5T3R deleted the dpl-1056-create-location-fix branch April 9, 2024 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DPL-1056 Unable to create buildings in LabWhere [Could've]
3 participants