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.

Sign up for GitHub

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

Jump to bottom

Site Resource improvements #20557

Closed
bryanboza opened this issue Jun 17, 2021 · 1 comment · Fixed by #20598
Closed

Site Resource improvements #20557

bryanboza opened this issue Jun 17, 2021 · 1 comment · Fixed by #20598
Labels
Changelog: Documented LTS: Excluded Ticket that has been excluded from at least one LTS Merged QA : Not Needed Release : 21.08 Type : Defect Type : Test Automation

Comments

@bryanboza
Copy link
Member

Related to #20393

  • We need to validate if the site already exists and return the correct message as we print in the log:
    Field [hostName] must be unique

  • Need to validate if we can catch the exception and print just a line error in case already exist another site with the same name. Right now we are too noisy printing the complete jersey error

  • Need to update all the postman test to use "SiteIdentifier" instead "hostIdentifier" variable

  • Need to add a postman test that try to delete the default site
@bryanboza bryanboza mentioned this issue Jun 17, 2021
Closed
@fabrizzio-dotCMS fabrizzio-dotCMS added this to the Scout Current milestone Jun 25, 2021
fabrizzio-dotCMS added a commit that referenced this issue Jun 25, 2021
@fabrizzio-dotCMS
#20557 site resource improvements
a402c3c
fabrizzio-dotCMS added a commit that referenced this issue Jun 28, 2021
@fabrizzio-dotCMS
#20557 - replacing ocurrences of host by site per QA request. Re-arra…
a1bb5df 
…nge GraphQL File Metadata Order to avoid tika issue
fabrizzio-dotCMS added a commit that referenced this issue Jun 29, 2021
@fabrizzio-dotCMS
#20557 reveting changes to graphQL postman test
8f1f204
@fabrizzio-dotCMS fabrizzio-dotCMS linked a pull request Jun 30, 2021 that will close this issue
Issue 20557 site resource improvements #20598
Merged
nollymar pushed a commit that referenced this issue Jun 30, 2021
@fabrizzio-dotCMS
Issue 20557 site resource improvements (#20598)
8ceae8c 
* #20557 site resource improvements

* #20557 - replacing ocurrences of host by site per QA request. Re-arrange GraphQL File Metadata Order to avoid tika issue

* #20557 reveting changes to graphQL postman test
@nollymar nollymar reopened this Jun 30, 2021
@wezell wezell closed this as completed Jul 13, 2021
@jcastro-dotcms jcastro-dotcms added the LTS : Next Ticket that will be added to LTS label Aug 3, 2021
jcastro-dotcms added a commit that referenced this issue Aug 16, 2021
@jcastro-dotcms
#20557 : Back-porting code changes to dotCMS 21.06.2 LTS release.
06a4561
@jcastro-dotcms jcastro-dotcms added LTS: Excluded Ticket that has been excluded from at least one LTS LTS: Released and removed LTS : Next Ticket that will be added to LTS labels Aug 16, 2021
@jcastro-dotcms
Copy link
Contributor

Changes were not back-ported to dotCMS 5.3.87.6 LTS because the code doesn't exist in there yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Changelog: Documented LTS: Excluded Ticket that has been excluded from at least one LTS Merged QA : Not Needed Release : 21.08 Type : Defect Type : Test Automation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue 20557 site resource improvements dotCMS/core
6 participants
@wezell @bryanboza @rweiner @nollymar @jcastro-dotcms @fabrizzio-dotCMS